Problem/Motivation

Like the issue #3133933: Send email on failure, I think that will be cool if send email in success cases.

CommentFileSizeAuthor
#7 Issue Evidence After Patch.png77.05 KBgabrielda
#2 3236380-2.patch6.26 KBlarisse

Issue fork backup_migrate-3236380

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:

About issue forks

Comments

larisse created an issue. See original summary.

larisse’s picture

larisse commented
Status: Active » Needs review
StatusFileSize
new 3236380-2.patch6.26 KB

A patch for this.

larisse’s picture

larisse commented
Assigned: larisse » Unassigned
takuma shimabukuro’s picture

takuma shimabukuro commented
Status: Needs review » Needs work

I couldn't apply the patch, please review the patch or recreate a new one for this issue

gabrielda’s picture

gabrielda
He/Him
commented
Assigned: Unassigned » gabrielda
gabrielda’s picture

gabrielda
He/Him
commented
Status: Needs work » Active
gabrielda’s picture

gabrielda
He/Him
commented
Assigned: gabrielda » Unassigned
Status: Active » Reviewed & tested by the community
StatusFileSize
new Issue Evidence After Patch.png77.05 KB

Hello,
I have tested the patch using Mail Log and could apply the patch and test the issue. It's correctly applied and I could reproduce both the error and correction.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 5.0.x-dev » 5.1.x-dev
Parent issue: #3133933: Send email on failure » #3223059: Plan for Backup and Migrate 5.1.0
Related issues: +#3133933: Send email on failure

Looks good.

  • DamienMcKenna committed 06039d1 on 5.1.x 
    Issue #3236380 by gabrielda, larisse, DamienMcKenna: Send email on...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

Committed. Thank you!

Status: Fixed » Closed (fixed)

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

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Follow-Up for missing config schema: #3475381: Config Schema is missing the Notify schema