warren
(Warren Buckley)
January 31, 2025, 7:23pm
1
Reading this
Seems I can mention a commit or perhaps an issue or PR from the repo for it to link nicely.
Seems like getting notifications in V15
v13/dev
← v13/bugfix/use-permissions-for-notifications
opened 06:46AM - 31 Jan 25 UTC
### Prerequisites
- [X] I have added steps to test this contribution in the d… escription below
Fixes: https://github.com/umbraco/Umbraco-CMS/issues/18167
### Description
Issue seemed to be caused by using a different method to add this action to the menu. When we do it the way we were, we have no `Action` property set, and hence the menu item is considered as one that's not affected by permissions.
Switching it to be registered in the way the other menu items were resolves this, and I don't see any downsides in doing so.
**To Test:**
- Verify that a user in a user group with the "Notifications" permission can manage notifications.
- Verify that a user in a user group without the "Notifications" permission cannot manage notifications.
sebastiaan
(Sebastiaan Janssen ⚓)
February 1, 2025, 12:42pm
2
I am excited about this plugin (haven’t added it yet) as it will ping a PR/issue if it’s being talked about on the forum. Additionally it will give badges to people for merged PR, that’s huge! Something I’ve been wanting automate on the old forum for years!
edit: plugin is active now, great stuff.
3 Likes