I see that there's no Drupal 8 version of this module, so I created a version of it - https://github.com/maijs/bcc I'd be happy to see a 8.x branch created from this code.

Some comments about Drupal 8 version:

  1. Email address is stored in key/value storage and not in configuration to allow per-environment settings without code change.
  2. There's no Enable Bcc checkbox. If email address is provided, it is used. If not, then it's not.

Comments

maijs created an issue. See original summary.

zoltán balogh’s picture

zoltán balogh commented
Status: Active » Reviewed & tested by the community

Thanks, its works well.

devkinetic’s picture

devkinetic commented

Your module does work, but putting this into key/value storage vs configuration I think would be a regression. There are a handful of ways to deal with multiple environments like config split or config override.

I used your code @maijs and refactored it.

  • Uses a dedicated settings form under config/system/
  • Uses a dedicated configuration yml
  • Added an enable/disable flag, akin to how the reroute email module works.

I export my production config to yml, and then use config override to set the enable flag to false for dev and staging. Before I upload, I need to fork your repo properly on github and put things into git there so credit goes to where credit is due.

Is there a maintainer active?

devkinetic’s picture

devkinetic commented

Here is my fork: https://github.com/devkinetic/bcc

Here is a pull request so you can see the comparison: https://github.com/maijs/bcc/pull/1

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Thank you @devkinetic,

you've been given access. Please care for the Drupal 8 / 9 branch and ensure it is Drupal 9 compatible and code-clean.

Thank you very much for your contribution!

devkinetic’s picture

devkinetic commented
Assigned: Unassigned » devkinetic
Status: Reviewed & tested by the community » Fixed

Thank you for the access @Anybody,

I've adding coding standards, a schema, and a basic test to get things started. I've also now created a 8.x-3.x branch and 8.x-3.0-beta1 release. I will do my best to review patches and welcome feedback as to what would earmark a stable release for D8.

Status: Fixed » Closed (fixed)

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