Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.It seems the file role_watchdog.permissions.yml is missing.
I'm not sure which was the idea, but in case it could be useful, I would suggest the following permissions for the module:
- 'administer role_watchdog' to allow access to the module settings form.
- 'access role_watchdog reports' to allow access to the user related views.
Thank you!
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 3153074-2.patch | 3.47 KB | gaurav.kapoor |
Comments
Comment #2
gaurav.kapoor CreditAttribution: gaurav.kapoor at Axelerant for Drupal India Association commented@markus_petrux I have added the permissions, modified the routes and views with appropriate permissions, added dependency in info.yml file as well as removed the unwanted routes.
Comment #3
gaurav.kapoor CreditAttribution: gaurav.kapoor at Axelerant for Drupal India Association commentedPlease have a look and update the issue with your suggestion. Once done, I'll push this patch as well.
Comment #5
gaurav.kapoor CreditAttribution: gaurav.kapoor at Axelerant for Drupal India Association commentedSince there have been some changes, please open up a new follow up issue(s) for your review and feedback. Thank you so much for working on this.
Comment #6
dwkitchen CreditAttribution: dwkitchen at CommitteeHQ commentedThis patch added a new dependency on "drupa:user"
Note the missing "l"
This is a breaking change
Comment #8
gaurav.kapoor CreditAttribution: gaurav.kapoor at Axelerant for Drupal India Association commented