Skip to content

Fix RSV and Pertussis routing if pregnant - #905

Open
benjystanton wants to merge 1 commit into
mainfrom
fix-rsv-pertussis-pregnancy-routes
Open

benjystanton wants to merge 1 commit into
mainfrom
fix-rsv-pertussis-pregnancy-routes

Conversation

@benjystanton

Copy link
Copy Markdown
Contributor
  • This fixes a bug that was causing the RSV and Pertussis interruption pages to not show
  • These should show conditionally, based on the estimated pregnancy date of the patient
  • Also, if you do the same patient different vaccination journey to record RSV and Pertussis, then the estimated pregnancy date is remembered, but the screen is not skipped (Not sure if this is what we want or if it can be improved?)

@benjystanton

Copy link
Copy Markdown
Contributor Author

@Anna-Sutton hi Anna, I think I've fixed the bug with the pregnancy date interruptions. Please could you check the journeys work as expected now please? (No rush)

@Anna-Sutton

Copy link
Copy Markdown
Contributor

@benjystanton I'm impressed that you got those warnings all lined up correctly.

The only thing I'm not sure about is the due date question still showing in the co-admin journey, albeit with the answers pre-populated. I was expecting not to see the question at all for the second vaccine but am now unsure of what we agreed with Ileri. Let's check that with her.

@benjystanton

Copy link
Copy Markdown
Contributor Author

@Anna-Sutton as you expected, Ileri also wants to skip the question entirely and go to consent unless a warning applies (in co-admin journey). So I'll take a look into adding this.

This branch was successfully deployed

1 active deployment
record-a-vac-fix-rsv-pe-hx9lkq — 0fe09f60 Deployed Oct 16, 2026 by frankieroberto
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.

3 participants