-
Notifications
You must be signed in to change notification settings - Fork 21
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
[Feat]: Notification entry navigates to the relevant view #2786
[Feat]: Notification entry navigates to the relevant view #2786
Conversation
Deployed to https://pr-2786.aam-digital.net/ |
@tomwwinter I have implemented logic for notification entries to navigate to the relevant view. Currently, we save the endpoint in the |
@sleidig Currently, I have implemented the logic for a dynamic URL in this way (you can check it in the file changes of this PR). It is working correctly in most cases, but for some entities, it is not generating the action URL because we are not getting the |
default: | ||
break; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
default: return notification.actionUrl?
Here is the new draft PR that updates the logic for redirecting the notification to the relevant view. |
closes: #2754
Visible/Frontend Changes