Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Fix favorites links #372

Closed
Noggling opened this issue Sep 26, 2023 · 2 comments · Fixed by #373
Closed

🐛 Fix favorites links #372

Noggling opened this issue Sep 26, 2023 · 2 comments · Fixed by #373
Assignees
Labels
frontend Frontend specific

Comments

@Noggling
Copy link
Contributor

Noggling commented Sep 26, 2023

Expected outcome

Short description

Add a trailing slash ("/") to the end of the link to ensure that the contextId is appended correctly.

Business Value / Developer Experience

Should give motivation for prioritization.
Outcome measurement of delivered value.@

Adding a trailing slash to links to ensure proper contextId appending is a small but impactful step that can contribute to various aspects of business success, including user satisfaction, data accuracy, cost reduction, brand reputation, conversion rates, and compliance.

Working Sorftware

User Acceptance Tests (UAT)

How to test

To prevent the application from crashing, please follow these steps:

  1. Add a pinned app.
  2. Click on the link.

Environments

All testing of issues are done in project-portal-test

For Backend use these

@Noggling Noggling self-assigned this Sep 26, 2023
@Noggling Noggling added the frontend Frontend specific label Sep 26, 2023
@Noggling Noggling linked a pull request Sep 26, 2023 that will close this issue
2 tasks
@AreWiv
Copy link
Contributor

AreWiv commented Sep 26, 2023

Tested and verified - deployed to Prod in Radix

@ken-mellem
Copy link
Contributor

@AreWiv then please set status to Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Frontend specific
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants