Skip to content

Fix RSV and Pertussis routing if pregnant - #905

Merged
benjystanton merged 3 commits into
mainfrom
fix-rsv-pertussis-pregnancy-routes
Oct 23, 2026
Merged

benjystanton merged 3 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
    • over 28 weeks - no interruption page will show

@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.

@frankieroberto
frankieroberto temporarily deployed to record-a-vac-fix-rsv-pe-hx9lkq October 21, 2026 11:53 Inactive
@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!

@Anna-Sutton

Copy link
Copy Markdown
Contributor

@benjystanton this all works a expected for me. But in the description at the top of this page, I'd delete these 2 bullets:

28 to 32 weeks - no interruption page will show
over 32 weeks - Pertussis interruption should show but the prototype doesn't handle this, yet

and instead say:

over 28 weeks - no interruption page will show

@frankieroberto
frankieroberto temporarily deployed to record-a-vac-fix-rsv-pe-hx9lkq October 23, 2026 16:06 Inactive
@benjystanton

Copy link
Copy Markdown
Contributor Author

@Anna-Sutton Thanks! I'll merge this in then

@benjystanton
benjystanton merged commit 6878178 into main Oct 23, 2026
1 check passed
@benjystanton
benjystanton deleted the fix-rsv-pertussis-pregnancy-routes branch October 23, 2026 16:09

This branch was previously deployed

1 inactive deployment
record-a-vac-fix-rsv-pe-hx9lkq — ebb3ad85 Deployed Oct 23, 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