v4.2.2 (work in progress) #4658
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
This fixes an issue where some vaccination records are currently invalid as they have both a `location_id` and a `location_name`. Jira-Issue: MAV-2016
Add `data_migrations:fix_vaccination_records_location`
These have all been run in our deployed environments so we can safely remove the migrations.
Remove Good Job migrations
This renames the `PatientSession` model to `PatientLocation` in preparation for the next commit which will restructure the model to remove the association to the session and instead link to a location and an academic year. Jira-Issue: MAV-1822
This reworks the `PatientLocation` model to no longer have a direct association with a session and instead have a separate `location_id` and `academic_year`, which represents the equivelant information since that's all that is needed to get the sessions for the patient. Jira-Issue: MAV-1822
This simplifies the lookups for patients and sessions by removing the duplicate scopes between patient locations and patients, instead using the scopes on the patient only, and making sure to filter out patients who shouldn't belong in the session before we need to query for this. Jira-Issue: MAV-1822
- Prevent frequently querying database for infrequently changing results
Reduce duplicate patient sessions
Introduce a rails cache for web server
JIRA Ticket: MAV-2006
…less_redis Add serverless elasticache deployment permissions
- Was only endpoint address but lacking protocol
This will be used to configure Rails caching using Redis. Jira-Issue: MAV-2006
This enables the use of Redis as a backend for the Rails cache, allowing for the cache to be distributed across the various web instances. Jira-Issue: MAV-2006
This adds a service class which is responsible for displaying counts in the headers and navigation bar. At the moment, this just refactors the existing code to use this class, but it allows us in the future to cache these values and improve performance. Jira-Issue: MAV-2006
This updates `TeamCachedCounts` to cache the values in the header to ensure a more performant page load instead of needing to query the latest information from the database each time. Jira-Issue: MAV-2006
Cache navigation header counts
- For invalidating caches JIRA Ticket: MAV-2006
This should have an improvement on performance, avoiding the need to compare the entire result set. Due to the size of the set, it's unlikely to be able to use a hash and instead performs a sequential scan. Jira-Issue: MAV-2004
Use `EXISTS` instead of `IN`
This makes a number of small improvements to the queries to improve performance and fix an issue with the session counts.
|
benilovj
approved these changes
Sep 18, 2025
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 merges the changes in 4.2.2 in to the
nextbranch now that we've merged 4.2.1 in tomain. Includes:data_migrations:fix_vaccination_records_location#4646EXISTSinstead ofIN#4649