Fix broken Master Ticket process with WalletConnect on L1#1129
Fix broken Master Ticket process with WalletConnect on L1#1129shawntobin wants to merge 1 commit into
Conversation
useWalletConnect.ts: - Removed PeerMeta state variable to reduce some state complexity since it’s not used outside of the hook definition anyway. I noticed an issue where it sometimes was returning null when it should have been set, so rather than put in a fix for this it might as well just be removed IMO. Deriving from ‘session’ when needed works as expected. ResetExecute.tsx: - Removed the WalletConnect session check from within performReticket and instead now using 2 separate useEffects to ensure correct execution based on wallet type. Other login methods shouldn’t need to wait for ‘connector’ and ‘session’, which are specific to WalletConnect. WalletConnect.tsx Metamask.jsx - Deleted these components since they’re not being used. Bridge no longer renders a tab screen for Metamask or WalletConnect. Tested successfully on mainnet with Zerion and Ledger.
|
lmfao |
|
here in 2026:
|
fixes #1124
Changes:
ResetExecute.tsx:
useWalletConnect.ts:
WalletConnect.tsx
Metamask.jsx
Tested successfully on mainnet with Zerion and Ledger.