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

Migrate legacy_acp asset failed. #360

Open
yanguoyu opened this issue Feb 28, 2024 · 1 comment
Open

Migrate legacy_acp asset failed. #360

yanguoyu opened this issue Feb 28, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@yanguoyu
Copy link

There are some problems with migrating legacy_acp.

  1. Asset accounts will not show legacy_acp, so I guess we can remove the migrate dialog and the user can migrate it to a new ACP asset account on Customized Assets page. What do you think? @Keith-CY
image
  1. Migrate legacy_acp failed on Customized Assets page.
migrate-failed.mov
  1. If there is only one address for the hard wallet and exist an asset account cell, we should better disable migration to a new asset account.
exist-asset-account.mov
@yanguoyu
Copy link
Author

yanguoyu commented Mar 6, 2024

nervosnetwork/neuron#3062
Neuron support migrates to other wallet ACP accounts, so even if the current one has an ACP account, users can choose to migrate to exist. So the third point will not change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: ✅ Done
Development

No branches or pull requests

2 participants