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

Add review page to navigator #210

Merged
merged 8 commits into from
Dec 14, 2020
Merged

Add review page to navigator #210

merged 8 commits into from
Dec 14, 2020

Conversation

jasalisbury
Copy link
Contributor

Description of change

The review page is the last page of the navigator. It does not use the navigator form component like the other pages of the form. It only shows nav state when the form as been submitted.

Note

This PR adds the mechanism for adding a review page. The current review page is a placeholder. There will be an additional PR with the review page as designed later. The hope is splitting up the PRs a bit makes for an easier review. If you feel we need to wait until the designed page is in place feel free to close without merging!

How to test

  1. Pull down changes
  2. Go to the review section of the activity reports, the submit button is disabled
  3. Completely fill out the form, clicking "continue" on every page
  4. The submit button is no longer disabled
  5. Click submit, note the state of the review page in the "side nav"
  6. Go to the first page and unselect all grantees
  7. The submit button is disabled

Compare against the desgin of the side nav.

Issue(s)

Checklist

  • Meets issue criteria
  • Code tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • [n/a] Documentation updated

@@ -9,7 +9,7 @@ import { Form as UswdsForm, Button } from '@trussworks/react-uswds';
import { useForm } from 'react-hook-form';

function Form({
initialData, onSubmit, onDirty, saveForm, renderForm,
Copy link
Contributor

Choose a reason for hiding this comment

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

Love using onContinue here, very clear that we're not submitting the whole process.

@rahearn rahearn merged commit 39a5461 into HHS:main Dec 14, 2020
rahearn added a commit that referenced this pull request Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants