- 🇩🇪Germany Anybody Porta Westfalica
2 Tests are failing. @Grevil could you take a look?
- 🇩🇪Germany Grevil
For some reason, "administer CAPTCHA settings" isn't inside the admin permission array. I'll fix it.
- last update
over 1 year ago 3 pass - Status changed to Needs review
over 1 year ago 9:57am 31 July 2023 - last update
over 1 year ago 3 pass - 🇩🇪Germany Grevil
Ok, I actually adjusted it once against, to be 100% sure, the right permission is used for the settings form.
- Assigned to webflo
- Status changed to RTBC
over 1 year ago 10:14am 31 July 2023 - 🇩🇪Germany Anybody Porta Westfalica
Thanks @Grevil! RTBC! Code LGTM, I don't think we should make changes in an update hook here, we *could* just inform users about the changed permission in an update hook notification (comment), without technical effect.
Should we do that? Leaving the question or commit to @webflo! :)
- last update
over 1 year ago 3 pass - Assigned to Anybody
- Status changed to Needs review
over 1 year ago 8:32am 26 October 2023 - 🇩🇪Germany Grevil
Please review again! We forgot to revoke the old permission. I created an appropriate update hook. webflo seems to be busy, so after review and manual testing of the hook we should merge this.
- Assigned to Grevil
- Status changed to Needs work
over 1 year ago 8:44am 26 October 2023 - 🇩🇪Germany Anybody Porta Westfalica
Well done @Grevil!
Could you add a simple permission testcase for the settings page so we can be 100% safe it works after this change?
Hopefully that's easy?RTBC once it goes green!
- last update
over 1 year ago 4 pass - Issue was unassigned.
- Status changed to Needs review
over 1 year ago 8:54am 26 October 2023 - 🇩🇪Germany Grevil
Done, let's wait for them to be green. Then we can merge.
- Status changed to RTBC
over 1 year ago 9:23am 26 October 2023 - last update
over 1 year ago 4 pass - Status changed to Fixed
over 1 year ago 9:41am 26 October 2023 Automatically closed - issue fixed for 2 weeks with no activity.