Problem/Motivation

After the backup succeeds, log a message.

Proposed resolution

After the backup succeeds, log a message.

Remaining tasks

After the backup succeeds, log a message.

User interface changes

A message would be logged after it succeeds.

API changes

TBD

Data model changes

TBD

CommentFileSizeAuthor
#10 After-patch-apply-3357357.png693.43 KBnikhil_110
#3 backup_migrate-n3357357-3.patch1.1 KBdamienmckenna
#2 backup_migrate-n3357357-2.patch839 bytesdamienmckenna

Issue fork backup_migrate-3357357

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

DamienMcKenna created an issue. See original summary.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new backup_migrate-n3357357-2.patch839 bytes

Another simple change. Again, not sure if this is the best place for it.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new backup_migrate-n3357357-3.patch1.1 KB

Also log afterBackup.

Status: Needs review » Needs work

The last submitted patch, 3: backup_migrate-n3357357-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

shivam_tiwari’s picture

shivam_tiwari commented
Assigned: Unassigned » shivam_tiwari

Akshay kashyap made their first commit to this issue’s fork.

Akshay kashyap opened merge request !30

shivam_tiwari’s picture

shivam_tiwari commented
Assigned: shivam_tiwari » Unassigned
akshay kashyap’s picture

akshay kashyap commented
Status: Needs work » Needs review
nikhil_110’s picture

nikhil_110 commented
StatusFileSize
new After-patch-apply-3357357.png693.43 KB

The above MR is applied successfully and working fine as expected. Attaching screenshots for reference.

  • Setup Drupal 10.x with Backup and Migrate Module
  • Apply MR
  • Go to Administration > Configuration > Development > Backup and Migrate > Backup now
akshay kashyap’s picture

akshay kashyap commented
Status: Needs review » Reviewed & tested by the community
mahima_mathur23’s picture

mahima_mathur23 commented
Priority: Normal » Minor
Issue tags: +error log
ivnish’s picture

ivnish
he/him
commented
Category: Task » Feature request
Issue tags: -error log

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

danrod’s picture

danrod
Location Ottawa
commented

I tested this and I saw the log after creating a backup, works as expected, thank you!

Merging this to the 5.1.x branch.

danrod’s picture

danrod
Location Ottawa
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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