- Issue created by @adamps
- Status changed to Needs review
over 1 year ago 10:14am 13 October 2023 - last update
over 1 year ago 32 pass, 6 fail The last submitted patch, 2: simplenews.view-subscriptions.3393892-2.patch, failed testing. View results →
- Status changed to Needs work
over 1 year ago 10:25am 13 October 2023 - 🇬🇧United Kingdom adamps
Hmm, but that makes in unavailable for users who only have "administer simplenews subscriptions". Really we'd like to work with both, but can't see how.
- First commit to issue fork.
- Merge request !66Issue #3393892: Subscribers view should be accessible with "view subscriptions" permission → (Merged) created by davps
- Status changed to Needs review
7 months ago 8:05pm 14 September 2024 @adamps, we can use own access plugin for views. MR !66 contains this solution.
- Status changed to Needs work
7 months ago 8:43am 19 September 2024 - 🇬🇧United Kingdom adamps
Neat, thanks. The tests are failing, it needs a schema.
I added the scheme. Other tests are fixed in the 🐛 Fix tests for 4.x branch Active .
- 🇬🇧United Kingdom adamps
Thanks I committed the other one so please rebase so we can see the tests passing.
Automatically closed - issue fixed for 2 weeks with no activity.