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

Design added to login page, Add report "readOnly" mode and Form navigation update #334

Merged
merged 36 commits into from
Mar 2, 2021

Conversation

jasalisbury
Copy link
Contributor

@jasalisbury jasalisbury commented Mar 1, 2021

67

Description of change

Make the login page look like the design:

  • Implement welcome placard, per design
  • Update tests that rely on login placard
  • Give that login placard the role=dialog.

How to test

Try both logging out, and changing BYPASS_AUTH=false. Login page should look like the design.

Issue(s)

Checklist

  • Meets issue criteria
  • Code tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • Documentation updated

285

Description of change

Reports are now placed in a read only mode if

  • The report's status is submitted or approved
  • The user viewing the report is NOT the author or a collaborator

In read only mode the user can only view the review page.

Also added a button/API call to move a report from submitted back to draft mode, allowing the author or collaborators to make edits after submitting but before approval.

How to test

  1. Pull down
  2. Fill out and submit an activity report
  3. Note after submitting you see the submitted review page and you cannot browse to other pages of the report
  4. Bring the report back to draft status
  5. Submit the report again
  6. As the approving manager pull up the report, note you are forced to stay on the review page, set the status to Needs Action
  7. As the author pull up the report, note you can edit the report as if it had a status of draft
  8. Resubmit the report and approve with the approving manager
  9. Note the report cannot be edited once approved

Keeping in draft until a few of the other report PRs get merged. Still need to update API docs with the new route.

Issue(s)

Checklist

  • Meets issue criteria
  • Code tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • Documentation updated

313

Description of change

  • Any page navigation saves the form to the API
  • Continue is not blocked by an invalid form
  • If the page is valid on navigation that page is marked as complete
    *~ Any save with an id of "new" triggers a page reload with the saved activity report's id instead of "new"~
  • Any save with an id of "new" will trigger a full page reload the next time the user navigates to a different page.
  • After visiting the review page validation errors are shown in the form
  • Fixed a bug where report recipients couldn't be removed

How to test

  1. Pull down changes
  2. Create a new report. Click "Continue" without completely filling out the first page, you should navigate to the second page
  3. Visit the review page, and back to the first page. You should see required fields marked with errors
  4. Completely fill out the first page. Navigate away with the side nav. The state of the first page should be "Complete"

Issue(s)

Checklist

  • Meets issue criteria
  • Code tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • Documentation updated

jasalisbury and others added 30 commits February 17, 2021 14:23
Conflicts:
	frontend/src/components/Navigator/index.js
	frontend/src/pages/ActivityReport/Pages/Review/Approver/__tests__/index.js
	frontend/src/pages/ActivityReport/Pages/Review/Approver/index.js
	frontend/src/pages/ActivityReport/Pages/Review/Submitter/Draft.js
	frontend/src/pages/ActivityReport/Pages/Review/Submitter/__tests__/index.js
	frontend/src/pages/ActivityReport/Pages/Review/Submitter/index.js
	frontend/src/pages/ActivityReport/Pages/Review/__tests__/index.js
	frontend/src/pages/ActivityReport/Pages/Review/index.js
	frontend/src/pages/ActivityReport/Pages/index.js
	frontend/src/pages/ActivityReport/__tests__/index.js
	frontend/src/pages/ActivityReport/index.js
	src/routes/activityReports/handlers.js
 * Any page navigation saves the form to the API
 * Continue is not blocked by an invalid form
 * If the page is valid on navigation that page is marked as complete
 * Any save with an id of "new" triggers a page reload with the saved
activity report's id instead of "new"
 * After visiting the review page validation errors are shown in the
form
 * Fixed a bug where report recipients couldn't be removed
Conflicts:
	frontend/src/pages/ActivityReport/__tests__/index.js
	frontend/src/pages/ActivityReport/index.js
	src/services/activityReports.js
Conflicts:
	frontend/src/components/Navigator/index.js
	frontend/src/pages/ActivityReport/Pages/Review/index.js
	frontend/src/pages/ActivityReport/index.js
Properly set some default values. Last saved time shows up when
navigating from a new report to a saved report
@jasalisbury jasalisbury requested a review from rahearn March 1, 2021 22:57
@rahearn rahearn merged commit 506d06a into HHS:main Mar 2, 2021
kryswisnaskas pushed a commit that referenced this pull request Jun 24, 2021
…Landing

Ttahub 138 al tp survey on ar landing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants