Skip to content
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

Filter vaccination records by programme #2965

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

thomasleese
Copy link
Contributor

@thomasleese thomasleese commented Feb 12, 2025

This follows up on #2939 to ensure that vaccination records are filtered by the current programme. At the moment this is the first programme administered in a session but in the future this will come from a selection by the user.

At the same time, this also fixes a bug where vaccination records administered outside of the current session weren't shown on a patient's record. Now we show all vaccination records for a particular patient.

Trello Card

@thomasleese thomasleese added 🐞 bug Something isn't working ✨ feature New functionality labels Feb 12, 2025
@thomasleese thomasleese added this to the v1.5.0 milestone Feb 12, 2025
@thomasleese thomasleese force-pushed the patient-show-all-vaccinations branch from c26dd32 to 18f45b4 Compare February 12, 2025 18:27
@thomasleese thomasleese force-pushed the patient-show-all-vaccinations branch from 18f45b4 to 65d3630 Compare February 12, 2025 18:27
@@ -37,14 +37,14 @@ class Organisation < ApplicationRecord
has_many :patients
has_many :sessions
has_many :teams
has_many :vaccination_records, -> { kept }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

This follows up on #2939 to ensure that vaccination records are filtered
by the current programme. At the moment this is the first programme
administered in a session but in the future this will come from a
selection by the user.

At the same time, this also fixes a bug where vaccination records
administered outside of the current session weren't shown on a patient's
record. Now we show all vaccination records for a particular patient.
@thomasleese thomasleese force-pushed the patient-show-all-vaccinations branch from 65d3630 to 09f22c5 Compare February 13, 2025 16:14
@tvararu tvararu temporarily deployed to mavis-pr-2965 February 13, 2025 16:15 Inactive
@thomasleese thomasleese merged commit 05e3877 into v1.5.0-wip Feb 13, 2025
10 checks passed
@thomasleese thomasleese deleted the patient-show-all-vaccinations branch February 13, 2025 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working ✨ feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants