Skip to content

feat: support cancelling backup restore (WPB-19677) - #5245

Open
sbakhtiarov wants to merge 1 commit into
developfrom
feat/cancellable-backup-restore
Open

feat: support cancelling backup restore (WPB-19677)#5245
sbakhtiarov wants to merge 1 commit into
developfrom
feat/cancellable-backup-restore

Conversation

@sbakhtiarov

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-19677

What's new in this PR?

Issues

The Cancel button did not stop an in-progress multiplatform backup restore.

Causes (Optional)

Restore and backup jobs were not separated, and the UI had no cancellation lifecycle that waited for persistence and temporary-file cleanup.

Solutions

  • Enable Cancel only for multiplatform restores.
  • Add Cancelling and Cancelled states.
  • Wait for the active persistence page and cleanup to finish before dismissing.
  • Prevent late progress updates and analytics from reporting cancellation as success or failure.
  • Keep legacy restores non-cancellable.

Comment on lines +143 to +145
isRestoreCompleted -> stringResource(R.string.label_ok)
isCancelling -> stringResource(R.string.backup_label_cancelling)
canCancel -> stringResource(R.string.label_cancel)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we could replace these three booleans with enum representing current state 馃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants