Clicking a notification title in the task list does not redirect to the url

[kaitlinnewson]

Describe the bug
In a user's notifications tasks list, clicking the title of the notification is not redirecting to the correct url related to that notification

To Reproduce
Steps to reproduce the behavior:

1. Login as a user that has task notifications (e.g "dbarnes")
2. Open the tasks list in the upper right corner
3. Click the title of a notification
4. See that it is marked as read, but does not redirect to the appropriate url

What application are you using?
OJS, OMP or OPS main

PRs:

• pkp-lib: pkp/pkp-lib#10495 fix notification urls #10603

[kaitlinnewson]

I've resolved some errors on the PHP side (PR above), but there is still an issue with the redirect event on the JS side of things.

It seems related to changes made in the redirectRequested event handling - when clicking on the notification link, it does not reach the siteHandler where the redirect would take place in redirectToUrl. I noticed the redirect worked when commenting out this line in AjaxModalWrapper.vue:

https://github.com/pkp/ui-library/blob/a0ab8c6ea9394b58a0afdd20bea03cc0a4058664/src/components/Modal/AjaxModalWrapper.vue#L135

@jardakotesovec any suggestions for the best approach here?

[jardakotesovec]

@kaitlinnewson yeah I noticed this issue as well, thanks for filling it... hopefully will get to it sometime soon.

[jardakotesovec]

@kaitlinnewson Good observations - we have now good compatibility layers for legacy modals with 3.5 (instead of 'manual' opening of that modal) - so basically just leveraged that and that seems to fix the problem.

@Tribunal33 Should be addressed on main. Can you give it a try? Thank you!

[Tribunal33]

Both bell icon displays list of alert notifications and the links within the alerts page are functioning when selecting them. I would say that they are redirecting to old submission workflows, but I think it is out of scope for this particular ticket. @jardakotesovec what do you think can we close this? I checked OMP and OJS alert bell.

[jardakotesovec]

@Tribunal33 Oki, I added note to #10670 make sure I don't forget it when I get to the redirects and having correct links within apps.