Skip to content

fix: select wallet page is empty - #4883

Merged
guanbinrui merged 1 commit into
developfrom
fix/empty-wallet-list
Nov 23, 2021
Merged

fix: select wallet page is empty#4883
guanbinrui merged 1 commit into
developfrom
fix/empty-wallet-list

Conversation

@nuanyang233

Copy link
Copy Markdown
Contributor

closes #4878

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions
github-actions Bot temporarily deployed to pull request November 22, 2021 19:20 Inactive
@wenluomask

Copy link
Copy Markdown
Contributor

Checked.

Comment on lines +80 to 81
if (!account) currentMaskWalletAccountSettings.value = ''
if (account && EthereumAddress.isValid(account)) currentMaskWalletAccountSettings.value = account

@hanszhoou hanszhoou Nov 23, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if (!account) currentMaskWalletAccountSettings.value = ''
if (account && EthereumAddress.isValid(account)) currentMaskWalletAccountSettings.value = account
if (account && EthereumAddress.isValid(account)) {
currentMaskWalletAccountSettings.value = account
} else {
currentMaskWalletAccountSettings.value = ''
}

@guanbinrui
guanbinrui merged commit 49d07c3 into develop Nov 23, 2021
@guanbinrui
guanbinrui deleted the fix/empty-wallet-list branch November 23, 2021 10:33
yanzhihong23 added a commit that referenced this pull request Nov 24, 2021
* develop:
  fix: add variant: constained to some buttons (#4911)
  refactor: remove unused code (#4905)
  chore: introduce @masknet/web3-providers (#4903)
  fix: nft cache name (#4901)
  fix: select wallet page is empty (#4883)
  feat: remove swap settings in labs page (#4890)
  fix: dialog ui issues (#4881)
  feat: implement getFungibleAssets of flow chain (#4879)

# Conflicts:
#	packages/dashboard/src/pages/Labs/index.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]Connect Mask wallet didn't go to create/import wallet page in popup

5 participants