Skip to content

Fix RSV and Pertussis routing if pregnant - #905

Open
benjystanton wants to merge 2 commits into
mainfrom
fix-rsv-pertussis-pregnancy-routes
Open

benjystanton wants to merge 2 commits into
mainfrom
fix-rsv-pertussis-pregnancy-routes

Conversation

@benjystanton

@benjystanton benjystanton commented Oct 16, 2026 •

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 added routing to eligibility page so it won't ask for estimated due date twice in co-admin journey, it will go straight to consent page, it will also show interruption pages if conditions are met…
    • before 16 weeks - both interruptions will show
    • 16 to 28 weeks - RSV interruption will show
    • 28 to 32 weeks - no interruption page will show
    • over 32 weeks - Pertussis interruption should show but the prototype doesn't handle this, yet

@frankieroberto
frankieroberto temporarily deployed to record-a-vac-fix-rsv-pe-hx9lkq October 16, 2026 16:28 Inactive
@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.

@benjystanton

Copy link
Copy Markdown
Contributor Author

@Anna-Sutton I've added in the extra routing to stop the estimated due date question being asked for co-admin routes. Could you review when you get a second please? No rush!

This branch was successfully deployed

1 active deployment
record-a-vac-fix-rsv-pe-hx9lkq — 7e592d9e Deployed Oct 21, 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