SIMSBIOHUB-287: User Access Request

[al-rosenthal]

Links to Jira Tickets

SIMSBIOHUB-287

Description of Changes

• Updated create access request to place email and display name in the correct fields
• Updated dev users that were affected by issue

Testing Notes

• Create a test BCeID
• Login to SIMS with your test BCeID and request access
• Login as admin user and approve the request
• Edit a project
• Open the Manage Team members autocomplete and see that the new user display name and email are consistent with other seed users

[github-actions]

Openshift URLs for the PR Deployment:

• App Route: https://biohubbc-app-1103-af2668-dev.apps.silver.devops.gov.bc.ca/
• Api Route: https://biohubbc-api-1103-af2668-dev.apps.silver.devops.gov.bc.ca/
• Pods: https://console.apps.silver.devops.gov.bc.ca/k8s/ns/af2668-dev/pods?name=1103

[codecov]

Codecov Report

Merging #1103 (aed7c5b) into dev (23374a8) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##              dev    #1103   +/-   ##
=======================================
  Coverage   63.28%   63.29%           
=======================================
  Files         531      531           
  Lines       15777    15776    -1     
  Branches     2438     2437    -1     
=======================================
  Hits         9985     9985           
+ Misses       5106     5105    -1     
  Partials      686      686           

Files	Coverage Δ
...repositories/administrative-activity-repository.ts	94.73% <ø> (ø)
...pi/src/services/administrative-activity-service.ts	100.00% <100.00%> (ø)
app/src/hooks/api/useAdminApi.ts	80.76% <ø> (+2.99%)	⬆️
app/src/hooks/useKeycloakWrapper.tsx	70.58% <ø> (ø)
api/src/paths/administrative-activity.ts	88.37% <66.66%> (ø)
app/src/features/admin/users/AccessRequestList.tsx	75.55% <50.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[curtisupshall]

Code looks good! I tested this in Openshift and it works as expected.

One minor bug that you may wish to address in this PR: when you submit your Access Request and are sent back to the home page, the home page doesn't show that you have a pending Access Request until you reload the page. We should peform a refresh of the user's pending access requests when they are sent ot the home page.

[NickPhura]

nit: can you update the params/doc for the getAdministrativeActivityStanding function in administrative-activity-service.ts. I think the param still says userIdentifier, rather than userGuid.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication