Page MenuHomePhabricator

[native] avoid showing SIWE backup message screen for non-primary devices
ClosedPublic

Authored by kamil on Nov 19 2024, 4:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 10, 1:41 PM
Unknown Object (File)
Mon, Feb 3, 9:34 AM
Unknown Object (File)
Mon, Feb 3, 8:43 AM
Unknown Object (File)
Mon, Feb 3, 7:23 AM
Unknown Object (File)
Wed, Jan 29, 11:30 AM
Unknown Object (File)
Wed, Jan 29, 9:11 AM
Unknown Object (File)
Sun, Jan 26, 4:43 AM
Unknown Object (File)
Sun, Jan 26, 4:13 AM
Subscribers

Details

Summary

ENG-9905.

This a fix for ENG-9917.

Depends on D13967

Test Plan

This is the final diff so testing this end-to-end.
I was scanning the QR code using my physical iOS device, secondary was the web and other iOS physical devices. I was testing this on both passwords and wallet users.
Things I checked:

  • Secondary logged in
  • Secondary received backup data (but ignored this for now)
  • On primary, the camera was dismissed and Secondary was added to the list
  • Tunnel broker connection (auth) created
  • Device list updates are sent

Things I didn't test (not sure if I should):

  • keyserver auth
  • scanning on Android (I don't have access to this device when creating this stack, I can do it later in the week)

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.Nov 19 2024, 4:28 AM
This revision is now accepted and ready to land.Nov 20 2024, 6:43 AM