Skip to content

Ensure currentUsername is set before comparing it with username#4750

Merged
emrysal merged 10 commits intomainfrom
fix-username-availability-issue
Sep 29, 2022
Merged

Ensure currentUsername is set before comparing it with username#4750
emrysal merged 10 commits intomainfrom
fix-username-availability-issue

Conversation

@hariombalhara
Copy link
Copy Markdown
Member

What does this PR do?

Fixes #4749

Environment: Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Impersonate a user and refresh multiple times the profile page to see the issue.

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 29, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Sep 29, 2022 at 2:50PM (UTC)

Copy link
Copy Markdown
Contributor

@alannnc alannnc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did once run on this error and looks like the fix worked.
NIT: @hariombalhara
On first render I'm able to see this text on yellow appears and then it's gone.
"You need to reserve your premium username for ..."
I don't know when it's triggering but can we add a flag to don't show? Till it's needed.

@emrysal emrysal merged commit 5e80f50 into main Sep 29, 2022
@emrysal emrysal deleted the fix-username-availability-issue branch September 29, 2022 16:05
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date core area: core, team members only

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

“Username already taken” message for accounts that own that username

5 participants