Skip to content

Conversation

jhenderson
Copy link
Contributor

@jhenderson jhenderson commented Sep 1, 2025

What's changed?

  • Cherry picked a commit from another PR to fix this bug
  • Updated the PatientSpecificDirectionsController to only add PSDs to patients consented for nasal spray along with updates to the feature spec to show that this behaviour now works as intended
  • Fix a styling issue on the bulk add page (screenshot below)
Screenshot 2025-09-01 at 15 18 45

@jhenderson jhenderson requested a review from a team as a code owner September 1, 2025 14:20
@jhenderson jhenderson force-pushed the fix-delegation-psd-bulk-add branch 2 times, most recently from c58b589 to 129d7fc Compare September 1, 2025 14:39
thomasleese and others added 3 commits September 1, 2025 15:55
This will allow code to filter on users who have a particular consent
status for a specific vaccine method.

Jira-Issue: MAV-1847
Recent changes to the gov-uk frontend framework has caused the cancel
link to sit underneath the button. This change brings the link back to
where it should be, on the right side of the button with some margin in
between.
We came across a bug where bulking adding PSDs would end
up including patients who were consented for injection only.
PSD's are only for HCAs to carry out nasal vaccinations.
@jhenderson jhenderson force-pushed the fix-delegation-psd-bulk-add branch from 129d7fc to acf1b7e Compare September 1, 2025 14:55
Copy link

sonarqubecloud bot commented Sep 1, 2025

@thomasleese thomasleese added the design tweak Small update to content or layout label Sep 1, 2025
@thomasleese thomasleese added this to the v3.4.0 milestone Sep 1, 2025
@jhenderson jhenderson merged commit 4c4d3eb into next Sep 1, 2025
13 checks passed
@jhenderson jhenderson deleted the fix-delegation-psd-bulk-add branch September 1, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design tweak Small update to content or layout
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants