- Issue created by @jOpdebeeck
- @jopdebeeck opened merge request.
- Status changed to Needs review
about 1 year ago 9:22am 20 October 2023 - 🇧🇪Belgium jOpdebeeck
I created a D10 compatible branch that we are currently using on our project in order to upgrade to D10.
Current changes on that branch can be reviewed already.
In the meantime people that are using this module can use the 3393651-d10-compatibility branch to upgrade their site to D10.
- 🇦🇹Austria agoradesign
thank you! that's better than nothing, but nevertheless it would be important, that the maintainer of this project could commit this into dev
btw, as I see that you're doing a bunch of code style fixes in your MR: you've missed out that single and redundant ";" in ConfirmClearSubscribers (you have moved that into a single line)
- 🇧🇪Belgium jOpdebeeck
@agoradesign Thx! I fixed this and also added an accessCheck I missed on the entityQuery in the WebPushNotificationController in commit 112133ad
-
skoro →
committed 18c72d4b on 8.x-1.x authored by
jOpdebeeck →
Issue #3393651: D10 compatibility
-
skoro →
committed 18c72d4b on 8.x-1.x authored by
jOpdebeeck →