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)
Sun, Dec 22, 12:25 AM
Unknown Object (File)
Sun, Dec 22, 12:24 AM
Unknown Object (File)
Sun, Dec 22, 12:24 AM
Unknown Object (File)
Sat, Dec 21, 4:04 AM
Unknown Object (File)
Thu, Dec 19, 12:39 AM
Unknown Object (File)
Mon, Dec 16, 2:47 PM
Unknown Object (File)
Mon, Dec 16, 12:12 PM
Unknown Object (File)
Mon, Dec 16, 2:42 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