-
-
Notifications
You must be signed in to change notification settings - Fork 156
Open
Labels
Ready for DevelopmentAutomatically assigned to issues that are ready to be picked upAutomatically assigned to issues that are ready to be picked upbugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomers
Description
Describe the bug
On the Bill Detail page, we require a user to have their Email verified before allowing them to submit testimony (and display a message to users without verified emails directing them to verify their email before submitting testimony).
It looks like navigating to the Submit Testimony form from another source (e.g. the Testimony Detail page's "Add Testimony" button) will skip this check and let users with unverified emails go through the publish testimony (it will eventually fail, but only at the point where they actually hit the "Publish" button, which is not an ideal UX).
To Reproduce
Steps to reproduce the behavior:
- Create a new user (and don't Verify your Email)
- Go to https://maple-dev.vercel.app/testimony/5PO5TI8lWo-dB8_rmKyWT/1
- Click on 'Add Testimony'
- See that it takes you to the Submit Testimony Page
Expected behavior
- Front-End changes:
- The "Add Testimony" button on the Testimony Detail page should have a state for users with Unverified Emails that prompts them to verify their email. This should function similarly to the Submit Testimony Card on the Bill Detail page
- The "Submit Testimony" page should have an initial step/modal pop up for unverified users prompting them to verify their emails before submitting (in case anyone gets directly linked to the Submit Testimony page)
- As a fallback, in case we miss another case in the future.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Ready for DevelopmentAutomatically assigned to issues that are ready to be picked upAutomatically assigned to issues that are ready to be picked upbugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomers