Repository navigation
Fix RSV and Pertussis routing if pregnant - #905
Open
benjystanton wants to merge 1 commit into
Open
benjystanton wants to merge 1 commit into
benjystanton wants to merge 1 commit into
Conversation
benjystanton
commented
Oct 16, 2026
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?)
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) |
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. |
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.