Skip to content

Conversation

@kris-szlapa
Copy link
Contributor

Summary

🎫 AEA-4755 PUT /selectedRole Fix
PR NHSDigital/eps-prescription-tracker-ui#384

  • Routine Change

Details

🧪 Regression tests for the selectedRole lambda fix

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 2, 2025

@kris-szlapa kris-szlapa merged commit d8eb497 into main Feb 3, 2025
5 checks passed
@kris-szlapa kris-szlapa deleted the AEA-4755-trackeruserinfo-selectedrole-fix branch February 3, 2025 11:06
kris-szlapa added a commit to NHSDigital/eps-prescription-tracker-ui that referenced this pull request Feb 3, 2025
## Summary

🎫 [AEA-4755](https://nhsd-jira.digital.nhs.uk/browse/AEA-4755) PUT
/selectedRole
🧪 Regression Tests:
NHSDigital/electronic-prescription-service-api-regression-tests#237
👤 Audrey Ricker: 555043304334
`CONFIDENTIAL: PERSONAL PATIENT DATA accessed by RICKER, Audrey -
General Medical Practitioner - NO_ORG_NAME (ODS: N82668)`
👤 Curtis Rogers: 555043308597
`CONFIDENTIAL: PERSONAL PATIENT DATA accessed by ROGERS, Curtis -
Registration Authority Manager - NO_ORG_NAME (ODS: A21293)`

- Routine Change



### Details of the Fix

- If the role with access matches `selectedRoleId`, it is added to the
`currently_selected_role` but not to the `roles_with_access` array. Move
the previously selected role back into rolesWithAccess, but only if it
was set.

- The RBAC (Role-Based Access Control) User Profile Banner follows these
patterns:
   - Standard user:
CONFIDENTIAL: PERSONAL PATIENT DATA accessed by LAST NAME, First Name -
RBAC Role - Site Name (ODS: ODS)
   - Locum user:
CONFIDENTIAL: PERSONAL PATIENT DATA accessed by LAST NAME, First Name -
RBAC Role - Locum pharmacy (ODS: FFFFF) - Site Name (ODS: ODS Code)

- The `confirmButtonText` contains the text: "Continue to find a
prescription"

---------

Co-authored-by: Sean Steberis <[email protected]>
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