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.

Sign up for GitHub

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

Jump to bottom

"Verify" button doesn't update right away #1501

Open
calebeby opened this issue Apr 19, 2022 · 2 comments
Open

"Verify" button doesn't update right away #1501

calebeby opened this issue Apr 19, 2022 · 2 comments
Assignees
@varCepheid
Labels
bug Addresses something that isn't working. help wanted More attention or people are needed. question Further information is requested.

Comments

@calebeby
Copy link
Member

It updates if you click it twice, or if you click it once and then refresh. This is confusing to users
@varCepheid varCepheid added bug Addresses something that isn't working. up for grabs Open for someone to work on. labels May 9, 2022
@varCepheid varCepheid self-assigned this Jun 3, 2022
@varCepheid varCepheid added wip and removed up for grabs Open for someone to work on. labels Jun 3, 2022
@varCepheid
Copy link
Collaborator

@calebeby After staring at the code for an hour, I have no idea what the issue is. In routes/user, I see a definition of fetchUser (which gets passed to VerifiedInfo as refetch) on line 428, and the definition of the button on line 103. I think the issue is coming from the refetch, but I don't have a good enough understanding of React Hooks to work with it or figure out where it's going wrong. It also may be coming from the clickability of Cards, but I don't think that's something we want to change. Can you take a look and see if there might be something I'm missing?

@varCepheid varCepheid added help wanted More attention or people are needed. question Further information is requested. and removed wip labels Jun 3, 2022
@varCepheid
Copy link
Collaborator

This problem also occurs for other user-profile changes. I attempted some unsuccessful changes in PR #1530 about dialogs for admins changing other users' profiles.

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

@varCepheid varCepheid

Labels
bug Addresses something that isn't working. help wanted More attention or people are needed. question Further information is requested.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@calebeby @varCepheid