38 lines
1.6 KiB
Markdown
38 lines
1.6 KiB
Markdown
---
|
|
title: "Remove wallet address verification popup — Quick Fix"
|
|
type: quick-fix
|
|
date: 2026-06-04
|
|
---
|
|
|
|
# Remove wallet address verification popup — Quick Fix
|
|
|
|
## Bug
|
|
|
|
The in-app-browser wallet address verification popup added friction and interfered with imToken login. The requested behavior is to remove that popup and keep the deeplink login flow direct.
|
|
|
|
## Root Cause
|
|
|
|
`AutoInjectedLogin` rendered a blocking confirmation dialog for `?autoLogin=` deeplink sessions before calling the injected wallet signature flow. That UI was unnecessary for the current wallet-login flow and could block or confuse imToken users.
|
|
|
|
## Fix
|
|
|
|
Removed the verification dialog and restored direct deeplink behavior: after the wallet in-app browser injects `window.ethereum`, the app calls `signInWithInjectedWallet()` and completes backend-verified login. Existing logged-in sessions still skip auto-login after stripping the deeplink parameter.
|
|
|
|
### Files Modified
|
|
|
|
- `src/wallet/AutoInjectedLogin.tsx` — removes the verification popup UI and auto-runs signature login for logged-out deeplink sessions.
|
|
- `src/wallet/injected.ts` — removes now-unused connected-address helper.
|
|
- `src/locales/zh-CN.ts` — removes unused verification popup copy.
|
|
- `src/locales/en.ts` — removes unused verification popup copy.
|
|
|
|
## Verification
|
|
|
|
- `rg -n "walletVerifyAddress|walletDetectedAddress|getConnectedInjectedAddress|wallet-verify" src || true`
|
|
- `npx tsc --noEmit`
|
|
- `npm run format:check`
|
|
- `npm test`
|
|
|
|
## Notes
|
|
|
|
The imToken injected-provider fallback remains in place; only the confirmation popup and its supporting copy/helper were removed.
|