I created a new patch, based on comment #10, compatible with commerce 2.36.
- Status changed to RTBC
8 months ago 3:46pm 6 May 2024 - last update
8 months ago 793 pass - 🇫🇷France nicolas bouteille
Thank you the patch works great for us on Commerce 2.38.
RTBC! - Status changed to Needs work
8 months ago 10:55am 9 May 2024 - 🇮🇱Israel jsacksick
Feels weird to expose all the different "operations" like this... What would be the usecase for using any of the emails not starting by "register_"?
Also, we use "\Drupal::configFactory()" instead of injecting the config factory. I get that this was probably done not to add an extra dependency breaking code decorating the login checkout pane, but this is not best practice.I wonder if we should expose the "human" labels instead of the operations machine names for the email options (See the attached screenshots):
Current options:
Suggestions: