Skip to content
This repository was archived by the owner on Sep 23, 2024. It is now read-only.

fix: account/network switch#73

Open
vang1ong7ang wants to merge 1 commit intoWalletConnect:masterfrom
vang1ong7ang:fix-account-network-switch
Open

fix: account/network switch#73
vang1ong7ang wants to merge 1 commit intoWalletConnect:masterfrom
vang1ong7ang:fix-account-network-switch

Conversation

@vang1ong7ang
Copy link
Copy Markdown

fix the account (or network) switch bug:

when activeIndex ( orchainId) is 0 the variable newActiveIndex (or newChainId) is not set properly

when `activeIndex` ( or`chainId`) is `0`, the variable `newActiveIndex` (or `newChainId`) is not set properly
@vang1ong7ang
Copy link
Copy Markdown
Author

to reproduce the bug:

  1. switch to another address
  2. try to switch back to the first address (index = 0): FAIL

@dusmart
Copy link
Copy Markdown

dusmart commented Feb 24, 2023

why is the repo maintainer so lazy, this is a great PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants