-
Notifications
You must be signed in to change notification settings - Fork 375
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
Comments
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
What version of Wallet are you using?
dev Version: 0.16.2
What is happening?
w
and replace it withw
, there's an error message stating the name is already in use.w
and replace it withW
, there's no error message and the wallet allows me to click ConfirmConfirm
produces an error message statingAccount Name Already Exists
Confirm
again shows no error message, as if the first click crashed the wallet (but I don't think it did)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
The text was updated successfully, but these errors were encountered: