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

Account name case sensitivity #1263

Open
HoTandy opened this issue Apr 30, 2024 · 1 comment
Open

Account name case sensitivity #1263

HoTandy opened this issue Apr 30, 2024 · 1 comment
Labels
bug Something isn't working p2

Comments

@HoTandy
Copy link
Collaborator

HoTandy commented Apr 30, 2024

What version of Wallet are you using?

dev Version: 0.16.2

What is happening?

  • I have 2 accounts in my wallet, I go to change the name of one of the accounts, I want to make the first letter of the word upper case rather than lower case.
  • If I delete the letter w and replace it with w, there's an error message stating the name is already in use.
  • If I delete the letter w and replace it with W, there's no error message and the wallet allows me to click Confirm
  • Clicking Confirm produces an error message stating Account Name Already Exists
  • Clicking Confirm again shows no error message, as if the first click crashed the wallet (but I don't think it did)
  • See video below to support the explanation above
Screen.Recording.2024-04-30.at.17.39.37.mov

steps to reproduce

See above and video attached

How it should behave?

  • The backend of the wallet name change functionality seems to be case insensitive as I cannot simply change a letter from lower case to upper case, it thinks the name is the same word

  • The frontend however spots it's a different name and doesn't show this on the wallet screen when I change a letter in the name to upper case rather than lower case

  • The frontend and back end should be consistent

  • Decide whether the naming conventions should be case sensitive or not, I believe it should be

What os are you seeing the problem on?

Mac/Apple Desktop

What browser?

Chrome

What is your browser version?

Version: 0.16.2

@HoTandy HoTandy added bug Something isn't working triage labels Apr 30, 2024
@LuizAsFight LuizAsFight added p2 and removed triage labels May 2, 2024
@BlackEyesForThree
Copy link

Additional explanation: The duplicate wallet name was modified again after encountering error messages, but no error messages were found

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working p2
Projects
None yet
Development

No branches or pull requests

3 participants