[native] unify username and wallet backup restore
ClosedPublic
Actions

Authored by kamil on Thu, Oct 31, 8:05 AM.

Details

Reviewers

varun
bartek

Commits

rCOMM5a8e0d6d32c6: [native] unify username and wallet backup restore

Summary

ENG-6145.

This allows us to decrease code duplication (next diff) and use the same interface for wallet/username users.

Additionally, moving restore methods to BackupMenu as this is for now only for testing purposes and we'll need something different for actual backup.

Depends on D13840

Test Plan

Tested in D13849

Diff Detail

Repository

rCOMM Comm

Branch

backup-work-5

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Thu, Oct 31, 8:05 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Oct 31, 8:05 AM

kamil retitled this revision from [native] unify SIWE and password user restore to [native] unify username and wallet backup restore.Thu, Oct 31, 8:08 AM

kamil edited the summary of this revision. (Show Details)

kamil added a child revision: D13846: [CommCoreModule][native_rust_library] deprecate SIWE related backup code.Thu, Oct 31, 8:10 AM

Harbormaster completed remote builds in B32478: Diff 45511.Thu, Oct 31, 8:24 AM

kamil published this revision for review.Thu, Oct 31, 8:51 AM

kamil mentioned this in D13840: [backup-client] make `backupID` mandatory when restoring backup.

kamil edited the test plan for this revision. (Show Details)

varun accepted this revision.Tue, Nov 5, 7:15 AM

This revision is now accepted and ready to land.Tue, Nov 5, 7:15 AM

bartek accepted this revision.Wed, Nov 6, 12:38 AM

Closed by commit rCOMM5a8e0d6d32c6: [native] unify username and wallet backup restore (authored by kamil). · Explain WhyThu, Nov 7, 4:07 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM5a8e0d6d32c6: [native] unify username and wallet backup restore.