Page MenuHomePhabricator

[backup-client][native] refactor restoring logic and deprecate `restoreBackup`
Needs ReviewPublic

Authored by kamil on Fri, Nov 22, 2:56 AM.
Tags
None
Referenced Files
F3367016: D14004.diff
Mon, Nov 25, 1:24 PM
Unknown Object (File)
Fri, Nov 22, 6:46 AM
Unknown Object (File)
Fri, Nov 22, 6:45 AM
Unknown Object (File)
Fri, Nov 22, 6:42 AM
Subscribers

Details

Reviewers
bartek
tomek
Summary

Part of ENG-9656

This was introduced mostly for testing purposes, there is no need to restore everything at once, we need to:

  1. Restore Backup Data on Secondary Device Auth
  2. Download User Keys to be able to decrypt data and sign a new device list using the old account, and then separately restore only Backup Data during the Restore protocol

We can test it as it should be (later in the stack), but this code is now hard to maintain, and keeping this adds a lot of additional and not needed work.

Depends on D14003

Test Plan

N/A, removing testing code and simplify logic, functionality tested in the next diff

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Fri, Nov 22, 4:01 AM
kamil edited the summary of this revision. (Show Details)
kamil added inline comments.
native/backup/restore-siwe-backup.react.js
40–50
native/profile/backup-menu.react.js
72–82