Problem/Motivation

We want to ensure that any non-production environment never ever sends email to real users and there are handy ways to test emails delivery on development environments.

Acceptance criteria

  • Email Reroute module is enabled and configured for all environments
  • Production environment has Email Reroute enabled, but ability to reroute emails is disabled in Email Reroute configs
  • Fallback to disable emails delivery when Email Reroute is not properly configured on non production environment is implemented

Tech concept

  • Install and enable Reroute email module
  • Add Reroute email as dependency in domino info.yml file
  • Add in composer.json of domino as required module for Reroute email module
  • Module should be installed but disabled in configs by default
  • For non production environments config should be overridden and rerouting functionality is enabled
  • Use Features module to export configs created as a part of this issue into Domino
  • Force disable email sending on non production environments if email reroute is not turned on & configured to reroute emails
  • Update documentation (README file + update module's description)

Issue fork domino-3312263

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

branimir.juranic created an issue. See original summary.

branimir.juranic’s picture

Issue summary: View changes
branimir.juranic’s picture

Issue summary: View changes
spleshka’s picture

Issue summary: View changes
spleshka’s picture

Title: Email delivery / reroute configuration » Email reroute feature

branimir.juranic’s picture

Assigned: branimir.juranic » Unassigned
Status: Active » Needs review
spleshka’s picture

Status: Needs review » Needs work

Thanks Branimir - I've reviewed the MR and left feedback what needs to be improved before the code can hit the main branch. Also, the MR is lacking hook_update_N() to automatically install email_reroute module if it's not installed yet on a project.

pflora made their first commit to this issue’s fork.

pflora’s picture

Status: Needs work » Needs review

Hello! I've done the changes requested by #8. I'm still new to Drupal so if there is anything that needs to be done better, please feel free to give me any kind of feedback and i'll try to work on it! :)

Moving this to NR.

spleshka’s picture

Status: Needs review » Needs work

Excellent job, @pflora, thank you so much! We're nearly there - there are just a few small description issues and a need to update CHANGELOG record. Then the issue is ready to be merged :)

spleshka’s picture

Status: Needs work » Fixed

Great job everyone, thanks so much!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.