USEPA / e-manifest

API services documentation and reference material for the e-Manifest hazardous waste tracking system
https://www.epa.gov/e-manifest
Creative Commons Zero v1.0 Universal
26 stars 22 forks source link

EM-2596 Add Handling for Selected Users with Changed Permissions #1503

Closed e-manifest closed 2 years ago

e-manifest commented 2 years ago

To account for users who’s permissions changed between when they’re initially selected/saved and being retrieved on the UI, add handling to not select those users/not crash.

Selected user and click Finish Later

!image-20220824-223401.png|width=1353,height=642!

Remove the users permission via User Maintenance

Access the manifest again

The table no longer displays the user, which is good since they no longer have access to the site

!image-20220824-223726.png|width=1381,height=651!

When clicking Finish Later or Review button, nothing happens and there’s a console error

!image-20220824-223556.png|width=1880,height=600!

Some handling should be added to throw out the registered user data who no longer have permission to a site.

Acceptance Criteria
(x) Previously selected registered users who have their permission removed are handled when the page loads and removed, so they can Review or Finish Later without issue