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.

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

Missing name on approved submitter users prevents them from submitting change requests #2606

Open
dakota002 opened this issue Oct 1, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@dakota002
Copy link
Contributor

When a submitter tried to submit a change request, but has not set their Name, they see the standard 403 error message. This is unclear, as it tells them to sign in when they already are.

Current behavior

The user sees a 403 error telling them to sign in

Expected behavior

If the user is already verified, they should be prompted to fill in their name and optionally their affiliation data before going to the edit page.

Steps to reproduce

Environment

Dev, Test, and Prod, any browser/OS

@dakota002 dakota002 added the bug Something isn't working label Oct 1, 2024
@lpsinger
Copy link
Member

lpsinger commented Oct 1, 2024

If the user is already verified, they should be prompted to fill in their name and optionally their affiliation data before going to the edit page.

Why do they need to fill in their name in order to submit a change request? Our code probably currently requires it, but maybe it shouldn't.

@dakota002
Copy link
Contributor Author

Yea, there is a check in the correction page that checks first if the user.name property is present, then if user.email is present. And if one is missing it just throws the same 403 error either way.

What do you think the proper behavior should be? I just threw this one together quickly after a ticket came in where a user experienced this

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

No branches or pull requests

2 participants