Page MenuHomePhabricator

[lib] attempt restoring User Data after restoring primary device
ClosedPublic

Authored by kamil on Thu, Apr 3, 5:00 AM.
Tags
None
Referenced Files
F6186659: D14533.id47695.diff
Wed, Apr 23, 11:13 AM
F6170951: D14533.id47701.diff
Wed, Apr 23, 2:08 AM
Unknown Object (File)
Sun, Apr 20, 12:33 PM
Unknown Object (File)
Fri, Apr 18, 4:22 PM
Unknown Object (File)
Fri, Apr 18, 6:20 AM
Unknown Object (File)
Thu, Apr 17, 2:20 PM
Unknown Object (File)
Mon, Apr 14, 11:10 PM
Unknown Object (File)
Mon, Apr 14, 10:56 AM
Subscribers

Details

Summary

ENG-10478

This is optional, just to start testing this.

Depends on D14532

Test Plan

Flip fullBackupSupport, delete app, install and restore - all thick threads are there

Diff Detail

Repository
rCOMM Comm
Branch
backup-publish
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Thu, Apr 3, 5:02 AM
kamil added inline comments.
native/account/restore.js
183

Looks like code duplication with D14532, but trying to implement one method makes it less readable

193

we should so it this way as this is the source of truth for now, but wondering about renaming this because this might be confusing

bartek added inline comments.
native/account/restore.js
193

Agree, but I don't have any ideas for a better name

This revision is now accepted and ready to land.Thu, Apr 3, 5:06 AM