search

status-im / status-mobile

a free (libre) open source, mobile OS for Ethereum
https://status.app
Mozilla Public License 2.0
3.88k stars 984 forks source link

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

Open mariia-skrypnyk opened 1 month ago

mariia-skrypnyk commented 1 month ago

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

mohsen-ghafouri commented 1 month ago

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 https://github.com/status-im/status-mobile/pull/20839, and as i checked with latest dev changes, couldn't reproduce the issue

mohsen-ghafouri commented 1 week ago

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