-
Notifications
You must be signed in to change notification settings - Fork 9
Store vaccine methods on consent forms #3673
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
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
c91ce92
to
662d944
Compare
1791745
to
eb714aa
Compare
efa5f59
to
c290f1c
Compare
eb714aa
to
96aa95a
Compare
c290f1c
to
1750686
Compare
96aa95a
to
b7505dc
Compare
b7505dc
to
606e36a
Compare
606e36a
to
de37e1f
Compare
de37e1f
to
47c77a6
Compare
jhenderson
approved these changes
Jun 6, 2025
1750686
to
0f773b2
Compare
47c77a6
to
84d0692
Compare
0caec24
to
f12e0ad
Compare
867d46f
to
75310c8
Compare
75310c8
to
02e914a
Compare
02e914a
to
ca0e12b
Compare
ca0e12b
to
368e49d
Compare
368e49d
to
c6653c3
Compare
8770c64
to
bf19acc
Compare
c6653c3
to
4dd588b
Compare
4dd588b
to
498f3b2
Compare
498f3b2
to
67c958a
Compare
This refactors the method to avoid duplication when generating the two types of consent (given and refused).
This will be used to store an array of validate vaccine methods on a consent, an enum backed by an array.
This adds a new column on consent forms (per programme) allowing parents to express a preference regarding the vaccine method. In most cases this is irrelevant as injection is the only option, but for Flu there's the option of a nasal spray which is the preferred choice. Jira-Issue: MAV-1230
This adds some validation on the `vaccine_method` column of the `Consent` model to ensure that it's always set when the parent has given consent. This value will be used as part of generating the overall consent status so it's important that it's always present. We can't validate that the `vaccine_method` is not set when refused consent because it's possible for a parent to give consent and then withdrawn it, in which case we would want to see what the method was when consent was given. Jira-Issue: MAV-1230
67c958a
to
a2d5ed8
Compare
|
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.
This adds a new column on consent forms (per programme) allowing parents to express a preference regarding the vaccine method. In most cases this is irrelevant as injection is the only option, but for Flu there's the option of a nasal spray which is the preferred choice.
To support this I've removed the old
contact_injection
column and related code as this is going to be rebuilt in the future as the designs have changed quite significantly since this was first added.Jira Issue