From df07e4e4f9be5097708a0f402271bd8f3909fda1 Mon Sep 17 00:00:00 2001 From: shuki Date: Fri, 6 Mar 2026 18:12:33 +0200 Subject: [PATCH] Remove duplicate completion notifications from backup/restore screens The app-level on_job_finished handler already shows these. Co-Authored-By: Claude Opus 4.6 --- tui/screens/backup.py | 12 ++---------- tui/screens/restore.py | 6 +----- 2 files changed, 3 insertions(+), 15 deletions(-) diff --git a/tui/screens/backup.py b/tui/screens/backup.py index 0afe88b..6c6a1ce 100644 --- a/tui/screens/backup.py +++ b/tui/screens/backup.py @@ -72,21 +72,13 @@ class BackupScreen(Screen): args = ["backup", f"--target={target}"] if remote: args.append(f"--remote={remote}") - rc = await job_manager.run_job(self.app, job, *args) - if rc == 0: - self.notify("Backup completed successfully", severity="information") - else: - self.notify(f"Backup failed (exit code {rc})", severity="error") + await job_manager.run_job(self.app, job, *args) @work async def _do_backup_all(self) -> None: job = job_manager.create_job("backup", "Backup All Targets") self.notify("Backup All started -- view in Running Tasks") - rc = await job_manager.run_job(self.app, job, "backup", "--all") - if rc == 0: - self.notify("All backups completed", severity="information") - else: - self.notify(f"Backup failed (exit code {rc})", severity="error") + await job_manager.run_job(self.app, job, "backup", "--all") def action_go_back(self) -> None: self.app.pop_screen() diff --git a/tui/screens/restore.py b/tui/screens/restore.py index 98cde28..03c7d30 100644 --- a/tui/screens/restore.py +++ b/tui/screens/restore.py @@ -152,11 +152,7 @@ class RestoreScreen(Screen): args.append(f"--dest={dest}") if skip_mysql: args.append("--skip-mysql") - rc = await job_manager.run_job(self.app, job, *args) - if rc == 0: - self.notify("Restore completed successfully", severity="information") - else: - self.notify(f"Restore failed (exit code {rc})", severity="error") + await job_manager.run_job(self.app, job, *args) def action_go_back(self) -> None: self.app.pop_screen()