bug: Swap page shows wallet not connected despite wallet being connected in header #7
Labels
No labels
agent:fix_bugfix
agent:fix_conflicts
agent:fix_security
agent:gap_analysis
agent:implement
agent:implement
agent:implement
agent:open_issues
agent:ready
agent:research
agent:security_audit
agent:verify
architecture
backend
blocker:hybrid
blocker:launch
blocker:limit-orders
blocker:v2
block:log_only
block:security
bug
ci
contracts
correctness
deploy
dev
devops
docs
documentation
duplicate
e2e
enhancement
epic
feature
frontend
functional-completion
gas
good first issue
governance
help wanted
high-risk
hooks
hybrid
indexer
infra
infrastructure
integrators
invalid
launch-blocker
limit-orders
localnet
localterra
low priority
missing-implementation
needs-design
ops
performance
priority
high
priority
medium
product
qa
QA
question
ready
ready
research
scripts
security
security-hardening
smartcontracts
tech-debt
testing
ux
UX
v2
verification
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
code/cl8y-dex-terraclassic#7
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Simulated dev wallet shows as connected in the header (address + Disconnect button visible), but the Swap page displays 'Wallet not connected. Please connect your wallet first.' and blocks swap execution.
The wallet connection state is not being shared properly between the header and the swap page component.
mentioned in issue #1
mentioned in issue #8
Fix committed (see commit on main)
Root cause: The simulated dev wallet path (
connectDev) was updating the Zustand store but never registering theMnemonicWalletin theconnectedWalletsmap. The header only checks the store, while the Swap page (and transaction layer) also checksgetConnectedWallet(), so they disagreed on connection state.Fix: Added
registerConnectedWallet()inwallet.tsand call it fromconnectDev()so the dev wallet is registered in the same map used by real wallets. Both the header and Swap page now see the wallet as connected.@Brouie Could you verify that the Swap page works correctly with the simulated dev wallet?
mentioned in commit
8b799f5f17Verified fixed — wallet state reaches swap page. Dev added registerConnectedWallet to wallet.ts.
mentioned in issue #162
mentioned in merge request !764
mentioned in merge request !828