From ee95eef395bcc3a27b3bd3108925ca6e38377c3c Mon Sep 17 00:00:00 2001 From: Aiden McClelland Date: Tue, 24 Mar 2026 19:19:47 -0600 Subject: [PATCH] fix: mark backup progress complete unconditionally MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove the backup_succeeded gate so the progress indicator updates regardless of the backup outcome — the status field already captures success/failure separately. --- core/src/service/transition/backup.rs | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/core/src/service/transition/backup.rs b/core/src/service/transition/backup.rs index 39e1a13e4..0b13df545 100644 --- a/core/src/service/transition/backup.rs +++ b/core/src/service/transition/backup.rs @@ -30,7 +30,6 @@ impl ServiceActorSeed { ErrorKind::Cancelled, )) }; - let backup_succeeded = res.is_ok(); let id = &self.id; self.ctx .db @@ -52,16 +51,14 @@ impl ServiceActorSeed { x => x, }) })?; - if backup_succeeded { - if let Some(progress) = db - .as_public_mut() - .as_server_info_mut() - .as_status_info_mut() - .as_backup_progress_mut() - .transpose_mut() - { - progress.insert(id, &BackupProgress { complete: true })?; - } + if let Some(progress) = db + .as_public_mut() + .as_server_info_mut() + .as_status_info_mut() + .as_backup_progress_mut() + .transpose_mut() + { + progress.insert(id, &BackupProgress { complete: true })?; } Ok(()) })