Add a new configurable email notification system

Actual functionality here works very well. No concerns with that. My nits are mainly about making this a better solution. The main one being around naming, which I recognize is a hard thing. Bumping back to NW.

1. +++ b/modules/role_expire_notification/config/schema/role_expire_notification.schema.yml
   @@ -0,0 +1,32 @@
   +    role_expire_notification_points:
   

   Calling this a notification point doesn't make as much sense as calling it a notification method. Let's rename this here and elsewhere.

2. +++ b/modules/role_expire_notification/role_expire_notification.info.yml
   @@ -0,0 +1,8 @@
   +core_version_requirement: ^8.8 || ^9
   

   Let's change this to also include Drupal 10.

Here's a fix for #15, along with a test of the new module. It also fixes several D10 incompatible issues. Mainly event dispatcher stuffs. It would be nice to have tests setup again as automated tests aren't executing right now.

@heddn I enabled testing.

9.5 and 10.1.x both passed green.

Re-rolled for 2.x. It was a trivial re-roll and the diff stats are identical.

And here's a 3.x patch.

@heddn none of your patches applies on Drupal 9. I am using Drupal 9.5.10.

@heddn @chike The patch cleanly applies to 3.x but this patch has to split:

1. Test fixes: See 🐛 Fix failing tests Fixed
2. New feature without test fixes

Once ready, this feature will be only committed to 3.x branch.

Removed changes that overlap with 🐛 Fix failing tests Fixed

any movement on this? it would be great to have this feature enabled or this module integrated with ECA for more power handling

Once drupal 11 version is released (new 4.x branch), a new MR should be created from current patch.

Added latest patch to a new issue fork and created a MR which targets 4.x branch.

Thanks for your contributions!

✨ Make Rules integration a sub-module, provide ECA integration Active would also be a good and more flexible alternative maybe?

@anybody I think both can live together. This issue it's already implemented, we should not force users to install modules if not necessary.

any movement on this? Would really be helpful to have email notifications

@elvin - albania drupal developer If you want to use it now you can patch the module via composer patching. It's fully working and it's multilingual. As you can see in the description and recent commits/messages, a lot of work has been done since your previous comment.

Thank you! Testing it right now and will report on anything i may find!

Thank you! Testing it right now and will report on anything i may find!