Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add useAccountName #6429

Merged
merged 11 commits into from
Jun 6, 2022
Merged

feat: add useAccountName #6429

merged 11 commits into from
Jun 6, 2022

Conversation

guanbinrui
Copy link
Member

@guanbinrui guanbinrui commented Jun 6, 2022

Description

In the new web3 infra, the wallet DB only serves for Mask wallets. We don't keep wallet records for external wallets anymore. In this PR, useAccountName is added for reading the name of external wallets.

  1. If the currently selected account is a Mask Wallet, use the name in the wallet record.
  2. Else, if an external wallet is used, the provider name will be the wallet name.

Closes # (NO_ISSUE)

Type of change

  • Documentation
  • Code refactoring (Restructuring existing code w/o changing its observable behavior)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (a fix or feature that would make something no longer possible to do/require old user must upgrade their Mask Network to this new version)

@guanbinrui guanbinrui marked this pull request as ready for review June 6, 2022 04:07
@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2022

@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 04:13 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 04:21 Inactive
@nuanyang233 nuanyang233 force-pushed the feat/use-account-name branch from bb1bb37 to 6fd115e Compare June 6, 2022 05:18
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 05:23 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 08:30 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 08:51 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 10:12 Inactive
@nuanyang233 nuanyang233 force-pushed the feat/use-account-name branch from 3cef71a to edce4ce Compare June 6, 2022 10:46
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 10:52 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 11:15 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 11:34 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 6, 2022 11:45 Inactive
@guanbinrui guanbinrui merged commit ce8864c into develop Jun 6, 2022
@guanbinrui guanbinrui deleted the feat/use-account-name branch June 6, 2022 12:29
etouyang added a commit to etouyang/Maskbook that referenced this pull request Jun 7, 2022
* 'develop' of github.com:etouyang/Maskbook: (173 commits)
  feat: more debugger connection actions (DimensionDev#6438)
  fix: sign tx by wallet with mnemonic words (DimensionDev#6437)
  fix: sign arbitrary message (DimensionDev#6436)
  fix: bugfix for refactor (DimensionDev#6434)
  fix: lost overrides (DimensionDev#6435)
  feat: add useAccountName (DimensionDev#6429)
  fix: button logic under EVM chain (DimensionDev#6431)
  fix: gas price fallback (DimensionDev#6432)
  chore: update node-version for workspace (DimensionDev#6428)
  refactor: social address list (DimensionDev#6410)
  fix: bugfix for trader (DimensionDev#6424)
  fix: duplicated db fetch (DimensionDev#6426)
  docs(i18n): sync i18n files from Crowdin (DimensionDev#6425)
  fix: tsc need restart to handle new file or removed file in watch mode
  fix: lint errors
  feat: referral farms (DimensionDev#6128)
  fix: fixup maskbox for evm migrate (DimensionDev#6395)
  fix: pets artblocks plugin (DimensionDev#6393)
  docs: update compat table
  refactor: add RecentTransactionComputed (DimensionDev#6420)
  ...

# Conflicts:
#	packages/dashboard/src/initialization/isolated_bridge.ts
#	packages/mask/src/extension/dashboard/index.tsx
#	packages/mask/src/plugins/Pets/SNSAdaptor/PetSetDialog.tsx
#	packages/mask/src/plugins/Pets/SNSAdaptor/PreviewBox.tsx
#	packages/mask/src/plugins/Pets/Services/rss3.ts
#	packages/mask/src/plugins/Pets/hooks/useNfts.ts
#	packages/shared/src/index.ts
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.

2 participants