Add unarchive patient functionality for support admins

[emyl3]

FRONTEND PULL REQUEST

• Resolves [Unarchive patient] Unarchive action #6064

Changes Proposed

• Adds unarchive patient functionality for support admins
• Adds a link to the "Unarchive patient" page on the support admin landing page

Additional Information

N/A

Testing

available on dev5 for testing

Screenshots / Demos

unarchive.mov

[fzhao99]

Code looks good overall! I found a minor bug testing in dev.

If you click fast enough into the facility dropdown after filling in the org selection, I think we might be hitting some sort of race condition for the facility selection not quite loading and causing us to get into a weird state in the form validation (notice the initial empty list on facility dropdown select that populates a second later). The result is after you fill in both of the dropdowns, you get "Org is required", even though it's selected. You can fix it just by hitting the search button again, so not too big of a deal, but flagging since it might be a bit confusing and I think the fix is pretty straightforward.

Screen.Recording.2023-08-28.at.11.00.48.AM.mov

In other places we pass a disabled prop based off the loading state of the org fetch.. I think if we do something similar here we can fix it.

[emyl3]

Code looks good overall! I found a minor bug testing in dev.

If you click fast enough into the facility dropdown after filling in the org selection, I think we might be hitting some sort of race condition for the facility selection not quite loading and causing us to get into a weird state in the form validation (notice the initial empty list on facility dropdown select that populates a second later). The result is after you fill in both of the dropdowns, you get "Org is required", even though it's selected. You can fix it just by hitting the search button again, so not too big of a deal, but flagging since it might be a bit confusing and I think the fix is pretty straightforward.

Screen.Recording.2023-08-28.at.11.00.48.AM.mov
In other places we pass a disabled prop based off the loading state of the org fetch.. I think if we do something similar here we can fix it.

OOO thank you for catching this -- will fix this!

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

82.1% Coverage
0.0% Duplication

[emyl3]

@fzhao99 ready for re-review!

[fzhao99]

👍

[BobanL]

Changes look good to me!

[johanna-skylight]

💯

[emyl3]

Thanks everyone! Will wait to hear back from @jayna-SkylightDigital before merging in. Leaving this as a note for myself so I don't accidentally merge it until the support team's approval 😅