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

Existing Account Address is overwritten if the same account is scanned with chains #20822

Closed
mariia-skrypnyk opened this issue Jul 19, 2024 · 4 comments
Assignees
Labels
wallet: Settings All issues for the mobile wallet settings
Milestone

Comments

@mariia-skrypnyk
Copy link

Bug Report

Steps:

  1. Login to app as User A
  2. Go to Profile -> Wallet -> Saved addresses
  3. Tap on + and paste address in such format 0xed620c9dc11401b63b35dfe8e93f2b11db0f6f9e with no chains.
  4. Save address as Test1
  5. Tap + and past the same address with added chain "eth:0xed620c9dc11401b63b35dfe8e93f2b11db0f6f9e"
  6. Save address as Test2

Actual behavior

No Test1 record in the list of Saved Addresses anymore, Test2 record is created as a new one

Expected behavior

Validation should take place when we try to create the Test2 record.

Screenshot 2024-07-19 at 16 10 16

https://www.figma.com/design/QWJV5WRbroXVDsdubEDjAT/Wallet-settings-for-Mobile?node-id=71-142051&t=EHg0EQkMykeEi6qM-0

Additional Information

  • Status version: nightly 19/07
  • Operating System: Android, iOS
@mohsen-ghafouri
Copy link
Contributor

Hey @mariia-skrypnyk, could you please check this issue if still reproducible or not, I belivee this issue already has been resolved in this PR #20839, and as i checked with latest dev changes, couldn't reproduce the issue

@mohsen-ghafouri mohsen-ghafouri moved this to Can't Reproduce in Status Desktop/Mobile Board Jul 31, 2024
@mohsen-ghafouri
Copy link
Contributor

Hey @mariia-skrypnyk in case you may missed my message, please check again this issue if it's reproducible anymore or not. thanks

@mariia-skrypnyk
Copy link
Author

Hey @mariia-skrypnyk in case you may missed my message, please check again this issue if it's reproducible anymore or not. thanks

Thanks @mohsen-ghafouri !
Issue is resolved, checked on nightly. 🙌

@mohsen-ghafouri
Copy link
Contributor

Thank you @mariia-skrypnyk.

@github-project-automation github-project-automation bot moved this from Can't Reproduce to Done in Status Desktop/Mobile Board Sep 9, 2024
@ilmotta ilmotta removed the bug label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wallet: Settings All issues for the mobile wallet settings
Projects
Archived in project
Development

No branches or pull requests

3 participants