Page MenuHomePhabricator

[landing] Reduce code duplication in landing/siwe.react.js
ClosedPublic

Authored by ashoat on Oct 9 2023, 10:09 AM.
Tags
None
Referenced Files
F3339743: D9432.diff
Thu, Nov 21, 8:34 PM
Unknown Object (File)
Wed, Nov 20, 6:57 PM
Unknown Object (File)
Wed, Nov 20, 4:39 PM
Unknown Object (File)
Wed, Nov 20, 4:39 PM
Unknown Object (File)
Wed, Nov 20, 4:37 PM
Unknown Object (File)
Wed, Nov 20, 4:18 PM
Unknown Object (File)
Wed, Nov 13, 9:54 PM
Unknown Object (File)
Wed, Nov 13, 9:49 PM
Subscribers

Details

Summary

This code is completely duplicated from lib/utils/wagmi-utils.js.

Test Plan
  1. In my local environment, make sure that mobile SIWE still works with Rainbow wallet (mobile SIWE uses landing, but web SIWE doesn't)
  2. In combination with other diffs in the stack, I tested the following on both my local environment, and in a staging environment for my keyserver on comm.domains:
    • Mobile sign in with Rainbow wallet using Rainbow login
    • Mobile sign in with Rainbow wallet using WalletConnect login
    • Web sign in with Rainbow wallet using Rainbow login
    • Web sign in with Rainbow wallet using WalletConnect login
    • Web sign in with Metamask wallet

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage