Skip to content

update error msg for user name change#1320

Merged
vnugent merged 1 commit intoOpenBeta:developfrom
mikeschen:update-name
Apr 16, 2025
Merged

update error msg for user name change#1320
vnugent merged 1 commit intoOpenBeta:developfrom
mikeschen:update-name

Conversation

@mikeschen
Copy link
Copy Markdown
Contributor


name: Pull request
about: Create a pull request
title: ''
labels: ''
assignees: ''


What type of PR is this?(check all applicable)

  • refactor
  • feature
  • bug fix
  • documentation
  • optimization
  • other

Description

Go to Edit Profile
Go to Username
Change username
Click Save
Go to change name again
See error msg (must wait 14 days)

Related Issues

Issue #460 (in OpenBeta repo)

What this PR achieves

Screenshots, recordings

Notes

@mikeschen mikeschen added the bug Something isn't working label Apr 7, 2025
console.error(e)
reset()
toast.error('Unexpected error. Please try again.')
toast.error(e instanceof Error ? e.message : 'Unexpected error. Please try again.')
Copy link
Copy Markdown
Contributor

@glassbead0 glassbead0 Apr 9, 2025

Choose a reason for hiding this comment

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

So does this correctly show an error message of Must wait 14 days to change username, instead of the unhelpful Unexpected Error message?

I see this issue on the backend, but it seems to me the 14 day wait is intentional, and this error message is the only thing that needs fixing. Should we close that backend issue, or am I missing something?

One thing that doesn't make sense to me is that I went to the deployment for this PR, and I was able to change the username several times in a row. I dont' see how this code change bypasses the 14 day limit 🤔

cc @mikeschen

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the FE component of the ticket.
I just put out PR for BE here: OpenBeta/openbeta-graphql#463

@vnugent vnugent merged commit 8e88d82 into OpenBeta:develop Apr 16, 2025
3 checks passed
@vnugent
Copy link
Copy Markdown
Contributor

vnugent commented Apr 16, 2025

Backend deployed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants