MokoConsulting/MokoSuiteBackup
1
0
Fork 0
Code Issues 28 Pull Requests Releases 2 Wiki Activity

fix: add success/progress messages to pre-update backup #192

Closed
jmiller wants to merge 2 commits from fix/preupdate-backup-messages into main
pull from: fix/preupdate-backup-messages
merge into: :main
Conversation 1 Commits 2 Files Changed 15
+24 -15
jmiller commented 2026-06-29 16:19:36 +00:00
Owner
Copy Link
Copy Source

Summary

  • Adds an info message ("in progress…") before the pre-update backup runs
  • Adds a success message ("completed successfully") when it finishes
  • Previously only failure was communicated to the admin

Test plan

  • Enable "Backup before update" in component settings
  • Update any extension
  • Verify green success message appears after backup completes
  • Verify warning message still appears on failure
## Summary - Adds an info message ("in progress…") before the pre-update backup runs - Adds a success message ("completed successfully") when it finishes - Previously only failure was communicated to the admin ## Test plan - [ ] Enable "Backup before update" in component settings - [ ] Update any extension - [ ] Verify green success message appears after backup completes - [ ] Verify warning message still appears on failure
jmiller added 2 commits 2026-06-29 16:19:37 +00:00
fix: add success/progress messages to pre-update backup
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Successful in 27s
Details
d1833bf06a 
Previously the pre-update backup only showed a message on failure.
Now shows an info message that the backup is in progress and a
success message when it completes.

Claude-Session: https://claude.ai/code/session_01MbEjBtsSjPuTWhqqrMS2wG
chore(version): pre-release bump to 01.45.08-dev [skip ci] 1e1d3a3ea9
jmiller commented 2026-07-06 00:31:14 +00:00
Author
Owner
Copy Link
Copy Source

Resolved and released (stable). The pre-update backup now surfaces an admin notification on every outcome — success, warning (local archive made but a remote upload failed), and failure — instead of only on failure. Implemented in plg_system_mokosuitebackup runPreActionBackup() via the BackupRunner status (PR #221).

Note: a live progress modal during the extension update (like "Backup Now") is separate and tracked in #196 — the update runs server-side in one request and can't drive the AJAX progress modal without injecting into com_installer's page.

Resolved and released (stable). The pre-update backup now surfaces an admin notification on **every** outcome — success, warning (local archive made but a remote upload failed), and failure — instead of only on failure. Implemented in `plg_system_mokosuitebackup runPreActionBackup()` via the `BackupRunner` status (PR #221). Note: a live *progress modal* during the extension update (like "Backup Now") is separate and tracked in #196 — the update runs server-side in one request and can't drive the AJAX progress modal without injecting into `com_installer`'s page.
jmiller closed this pull request 2026-07-06 00:31:34 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
component: admin
Admin UI and views
 component: api
REST API
 component: engine
Backup/restore engine
 component: remote
Remote storage (FTP/GDrive)
 component: scheduler
Scheduled tasks
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Priority -
Type -
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoSuiteBackup#192
Delete Branch "fix/preupdate-backup-messages"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?