Closed dhriaznov closed 8 months ago
Thanks for the review folks, I updated the PR, feel free to take a second look π
I just resolved the git conflicts, could you approve again pls? @fedeerbes
Test with build here: https://github.com/secretkeylabs/xverse-web-extension/actions/runs/7630950545
Fixed the issue DuΕ‘ana found, @fedeerbes could you please review?
tested can be merged π
π PR Type
What kind of change does this PR introduce?
π Background
As an Xverse user I want to see rename my accounts after Iβve selected them So that I can identify them easily when selecting & switching accounts
Issue Link: #[ENG-3036] Context Link (if applicable):
π Changes
validateAccountName
helper func and used it to validate the naming/renaming of ledger and regular accountsImpact:
πΌ Screenshot / πΉ Video
Videos are in the linear ticket.
β Review checklist
Please ensure the following are true before merging: