MokoConsulting/MokoSuiteBackup

fix(plugin): pre-update backup notifies on success, not only failure #221

Merged

jmiller merged 2 commits from fix/preupdate-notification into dev

2026-07-05 23:58:24 +00:00

Author	SHA1	Message	Date
mokogitea-actions[bot]	dfd4e77ade	chore(version): pre-release bump to 02.57.09-dev [skip ci] RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped Details Branch Cleanup / Delete merged branch (pull_request) Successful in 3s Details	2026-07-05 23:58:13 +00:00
jmiller	1a1c81550c	fix(plugin): pre-update backup notifies on success too, not only failure Universal: PR Check / Require Docs Update (pull_request) Has been skipped Details Universal: PR Check / Wiki Update Reminder (pull_request) Has been skipped Details Universal: PR Check / Branch Policy (pull_request) Successful in 3s Details Generic: Project CI / Lint & Validate (pull_request) Successful in 17s Details Universal: PR Check / Secret Scan (pull_request) Successful in 12s Details Universal: Pre-Release / Build Pre-Release (${{ inputs.stability \|\| github.ref_name }}) (push) Successful in 44s Details Universal: PR Check / Validate PR (pull_request) Failing after 1m5s Details Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Successful in 1m13s Details Generic: Project CI / Tests (pull_request) Has been cancelled Details Universal: PR Check / Build RC Package (pull_request) Has been cancelled Details Universal: PR Check / Report Issues (pull_request) Has been cancelled Details runPreActionBackup() only enqueued a message when the backup failed, so a successful pre-update backup produced no admin notification at all. Now it surfaces every outcome via the BackupRunner status: complete -> success, warning (archive made but remote upload failed) -> warning, fail -> error. Refs #192 Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 18:57:42 -05:00