Skip to content

Conversation

summer-cook
Copy link
Contributor

@summer-cook summer-cook commented Jan 30, 2024

Story

The request form should display both for logged in and logged out users. For users who are logged out, the form and button should be disabled.

Related

Expected Behavior Before Changes

Before this PR, a user clicked on a ware to make a request, and they would end up at a page with just a notice that said "Unauthorized"

Expected Behavior After Changes

After this PR, when a logged out user clicks on a ware to make a request, they are able to see the form to make the request, but it is disabled. They can also see a notice at the top of the form prompting them to log in if they would like to fill it out.
If the user is logged in, they should still be able to fill out the form and submit requests.

Screenshots / Video

Video: https://share.zight.com/NQuBr6Bx

Disabled form

screencapture-localhost-3000-requests-new-crispr-2024-01-30-14_55_53

Copy link

vercel bot commented Jan 30, 2024

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

Name Status Preview Comments Updated (UTC)
webstore-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 31, 2024 7:05pm

@alishaevn alishaevn merged commit 979b719 into main Jan 31, 2024
@alishaevn alishaevn deleted the 345-disable-request-form branch January 31, 2024 20:15
@alishaevn alishaevn linked an issue Feb 6, 2024 that may be closed by this pull request
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

show disabled request form for logged out users
3 participants