- Issue created by @alex.bukach
- 🇩🇪Germany jurgenhaas Gottmadingen
Great observation, thanks for reporting this @Alex Bukach
The current implementation of the EventView class has the purpose to mark notifications as seen when the notified user clicks on the event in the notification drop down view. Then they get redirected to the entity which triggered that event.
Later, we implemented new functionality that use the Event entity as a placeholder for events that are not directly related to any other content entity. And we missed the fact that for the notification rendering, this same method is being called. In that case, we don't want to mark the notification as seen, and we don't want to redirect. Instead, we need to build some rendering array. I'll see what I can do for that.
- Merge request !9Issue #3379118: Cron requests are redirected when DANSE Config, DANSE Form, DANSE Log, or DANSE User is enabled → (Merged) created by jurgenhaas
- Status changed to Needs review
over 1 year ago 7:48am 5 August 2023 - 🇩🇪Germany jurgenhaas Gottmadingen
Looks like it wasn't that difficult, please have a look at the MR and let me know if that resolves the issue for you.
- Status changed to RTBC
12 months ago 6:32pm 1 December 2023 -
jurgenhaas →
committed 304b4783 on 2.2.x
Issue #3379118 by jurgenhaas, Alex Bukach: Cron requests are redirected...
-
jurgenhaas →
committed 304b4783 on 2.2.x
-
jurgenhaas →
committed 64237e91 on 2.3.x
Issue #3379118 by jurgenhaas, Alex Bukach: Cron requests are redirected...
-
jurgenhaas →
committed 64237e91 on 2.3.x
-
jurgenhaas →
committed a8f5a2aa on 2.3.x
Issue #3379118 by jurgenhaas, Alex Bukach: Cron requests are redirected...
-
jurgenhaas →
committed a8f5a2aa on 2.3.x
- Status changed to Fixed
12 months ago 7:47am 4 December 2023 - 🇩🇪Germany jurgenhaas Gottmadingen
Thanks @Alex Bukach, this is now published in release 2.2.19 and ported to 2.3.x
-
jurgenhaas →
committed c32e8ac6 on 2.3.x
Issue #3379118 by jurgenhaas, Alex Bukach: Cron requests are redirected...
-
jurgenhaas →
committed c32e8ac6 on 2.3.x
Automatically closed - issue fixed for 2 weeks with no activity.