Problem/Motivation

From UpdateEmailBuilder, notification email address for update.settings is overridden with dummy value.

Update manager settings

But this way, it fails config validation with Configuration Inspector cause
dummy is not a valid email address.

cinsp error

Proposed resolution

We should use either valid email address or empty value for config_overrides section in UpdateEmailBuilder

 *   config_overrides = {
 *     "update.settings" = {
 *       "notification" = { "emails" = { "dummy" } },
 *     },
 *   },
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

sorlov created an issue. See original summary.

sorlov’s picture

Status: Active » Needs work
sorlov’s picture

Issue summary: View changes
StatusFileSize
new171.47 KB
adamps’s picture

Version: 1.4.1 » 2.x-dev
Issue tags: +Novice

We can change dummy to dummy@example.com. The code is now in UpdateOverride.php.

koustav_mondal’s picture

Assigned: Unassigned » koustav_mondal

Working on it.

koustav_mondal’s picture

Assigned: koustav_mondal » Unassigned
Status: Needs work » Needs review

@adamps Made the changes. Please review it.

adamps’s picture

Status: Needs review » Fixed

adamps’s picture

Thanks

Status: Fixed » Closed (fixed)

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