Page MenuHomePhabricator

[native] Let user skip ConnectEthereum screen if ethereumAccount already set
ClosedPublic

Authored by ashoat on Dec 20 2023, 1:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 27, 9:11 AM
Unknown Object (File)
Fri, Sep 27, 9:11 AM
Unknown Object (File)
Fri, Sep 27, 9:08 AM
Unknown Object (File)
Fri, Sep 27, 9:02 AM
Unknown Object (File)
Aug 28 2024, 2:41 AM
Unknown Object (File)
Aug 10 2024, 8:45 AM
Unknown Object (File)
Aug 10 2024, 8:45 AM
Unknown Object (File)
Aug 10 2024, 8:45 AM
Subscribers

Details

Summary

This modifies the ConnectEthereum screen so that if there is already a connected Ethereum account, we don't force the user to do the whole SIWE auth process again.

Depends on D10416

Test Plan

Here's what the updated screen looks like:

Screenshot 2023-12-20 at 5.00.26 PM.png (2×1 px, 872 KB)

Additionally, I tested this diff stack as follows:

  1. Delete test ETH account 1. Attempt to do main SIWE login with ETH account 1. Confirm it redirects to registration flow. Complete registration flow and confirm account successfully created
  2. Same as above, but go back after reaching the avatar screen, and then forward to see the Ethereum screen. Choose the option to use the already connected ETH account, then complete registration flow and confirm account successfully created
  3. Same as above, except in the Ethereum screen, choose the option to "skip" the Ethereum login. Confirm password account is successfully created
  4. Same as above, except in the Ethereum screen, select another Ethereum wallet that already has a Comm account. Confirm that we hit ExistingEthereumAccount, and that we're able to login directly with the existing account
  5. Same as above, except in the Ethereum screen, select another Ethereum wallet that doesn't already have a Comm account. Confirm that the account is successfully created using the second Ethereum wallet and not the first
  6. Same as above, except in the Ethereum screen, go through the auth flow again for the exact same wallet. Confirm that the account is successfully created

Note that I needed to apply this patch to work around ENG-6097.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable