Page MenuHomePhabricator

[native] upload User Keys backup when user is staff
ClosedPublic

Authored by kamil on Wed, Nov 27, 9:42 AM.
Tags
None
Referenced Files
F3514352: D14059.diff
Sun, Dec 22, 4:26 AM
Unknown Object (File)
Tue, Dec 17, 9:02 PM
Unknown Object (File)
Sun, Dec 15, 9:00 AM
Unknown Object (File)
Tue, Dec 10, 5:45 AM
Unknown Object (File)
Tue, Dec 10, 4:40 AM
Unknown Object (File)
Thu, Dec 5, 1:11 PM
Unknown Object (File)
Wed, Dec 4, 8:37 AM
Unknown Object (File)
Tue, Dec 3, 8:16 PM
Subscribers

Details

Summary

Part of ENG-9604 but only for staff.

This is also needed for ENG-9711.

Depends on D14056

Test Plan
  1. Check if backup is uploaded when user has no backup
  2. Log out/in and check if backup is uploaded

Diff Detail

Repository
rCOMM Comm
Branch
land
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Wed, Nov 27, 10:22 AM
kamil added inline comments.
native/backup/backup-handler.js
57 ↗(On Diff #46102)
97–98 ↗(On Diff #46102)

dispatchActionPromise is swallowing errors, this is to avoid that

native/backup/backup-handler.js
101 ↗(On Diff #46102)

Should we only show this alert to staff?

bartek added inline comments.
native/backup/backup-handler.js
101 ↗(On Diff #46102)

Looks like it's the same as in https://phab.comm.dev/D14056#inline-78946 - it's gated on staffCanSee

This revision is now accepted and ready to land.Fri, Nov 29, 2:56 AM
  • rebase
  • use useStaffAlert