-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 nickname is always defined #4684
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
Looks great to me! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Good work, just a small comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tommasini I found a fun edge case:
If I were to:
- long press to edit my account nickname
- remove the current name
- tap on the space bar to set ' ' as my account nickname
- upon doing so, I still experience the bug where I am unable to edit my account nickname after setting it to ' '
see recording
@cortisiko Really well catched, I was not considering the blank spaces!! Thank you Curtis! Awesome!!! @gantunesr Could you give me another review please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌮 🌮 looks good @tommasini
Description
The issue is that when the nickname of the account would be empty, it would be non editable.
Proposed Solution
The proposed solution it is, when the name it's empty we will assume that the name of the account will be the last name of it.
Example:
If it was "Account 2" after confirm the empty string on the account name, it will come back to be "Account 2"
Also its proposed, for the users that already have no nickname on their accounts we set the default name "Account" when the user opens the app.
Code Impact
Low
Test Cases
Case1:
Screenshots/Recordings
WhatsApp.Video.2022-07-18.at.6.54.45.PM.mp4
If applicable, add screenshots and/or recordings to visualize the before and after of your change
Issue
Progresses #4308
Checklist