- Issue created by @vincent signoret
- First commit to issue fork.
- Status changed to Needs review
12 months ago 10:55am 18 December 2023 - 🇮🇳India shalini_jha
hi @vincent signoret
I have fixed the issue and added a patch.please review
Thanks - 🇨🇦Canada b_sharpe
It looks like this is a result of 📌 Automated Drupal 10 compatibility fixes Fixed in which webpush isn't d10 ready, the patch isn't going to work as it still is missing. Instead of applying patches here, I think for now the best bet is to remove this dependency as it's only needed for
notification_system_dispatch_webpush
module in which case a user can work out a solution there. -
b_sharpe →
committed ba3aa9fd on 1.x
Issue #3409115 by shalini_jha, vincent signoret, b_sharpe: D10...
-
b_sharpe →
committed ba3aa9fd on 1.x
- Status changed to Fixed
11 months ago 5:54pm 11 January 2024 - 🇨🇦Canada b_sharpe
Removed and updated project page as well as 📌 Update README about Web Push Active
- 🇨🇦Canada vincent signoret
Thank you, @b_sharpe !
I believe this is the best approach. We can still install these dependencies if needed using https://github.com/mglaman/composer-drupal-lenient. Automatically closed - issue fixed for 2 weeks with no activity.