Skip to content

Conversation

thomasleese
Copy link
Contributor

@thomasleese thomasleese commented Sep 3, 2025

When a prescriber records verbal consent, we also ask the user if they would like to add a PSD to the patient. This wasn't working correctly, and instead a PSD wasn't being added to the patient. This is because the logic related to creating PSDs was specific to the triage journey that occurs outside of the verbal consent journey. Instead I've moved the logic in to the TriageForm ensuring that both triage journeys will have the same logic.

Jira Issue - MAV-1905

@thomasleese thomasleese added this to the v3.4.0 milestone Sep 3, 2025
@thomasleese thomasleese requested a review from a team as a code owner September 3, 2025 14:36
@thomasleese thomasleese added the bug Something isn't working label Sep 3, 2025
@thomasleese thomasleese force-pushed the ensure-psds-are-created branch 3 times, most recently from b43259f to 29d4ceb Compare September 3, 2025 15:05
When a prescriber records verbal consent, we also ask the user if they
would like to add a PSD to the patient. This wasn't working correctly,
and instead a PSD wasn't being added to the patient. This is because the
logic related to creating PSDs was specific to the triage journey that
occurs outside of the verbal consent journey. Instead I've moved the
logic in to the `TriageForm` ensuring that both triage journeys will
have the same logic.

Jira-Issue: MAV-1905
@thomasleese thomasleese force-pushed the ensure-psds-are-created branch from 29d4ceb to 7a723d7 Compare September 3, 2025 15:24
Copy link

sonarqubecloud bot commented Sep 3, 2025

Copy link
Contributor

@jhenderson jhenderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thomasleese thomasleese merged commit f91b637 into next Sep 3, 2025
13 checks passed
@thomasleese thomasleese deleted the ensure-psds-are-created branch September 3, 2025 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants