Skip to content

Conversation

aldavidson
Copy link
Contributor

@aldavidson aldavidson commented Sep 4, 2025

The model was added in #4484, with no means of populating it.

This PR adds a VaccinationRecord.create_or_update_reporting_api_vaccination_event method, and a Mavis CLI command bin/mavis reporting-api update-vaccination-events to call that method for all VaccinationEvents created after a given date time.

Future work could call the create_or_update_reporting_api_vaccination_event method in a suitable model callback, but this is enough for now. Closer to go-live, we could add a scheduled job to do this every night, or something similar.

@aldavidson aldavidson requested review from a team as code owners September 4, 2025 14:04
@aldavidson aldavidson changed the base branch from main to next September 4, 2025 14:04
@aldavidson aldavidson changed the title Add method and CLI command to generate ReportingAPI::VaccinationEvents Add method and CLI command to populate ReportingAPI::VaccinationEvents Sep 4, 2025
@aldavidson aldavidson force-pushed the add-method-and-cli-task-to-generate-vaccination-events branch from 8e22d10 to 9c7aa95 Compare September 4, 2025 15:17
@thomasleese thomasleese force-pushed the add-method-and-cli-task-to-generate-vaccination-events branch from 9c7aa95 to f2310f6 Compare September 5, 2025 16:52
… VaccinationRecords

linter-compliant version using Time.zone.parse after the .load_rails call
@thomasleese thomasleese force-pushed the add-method-and-cli-task-to-generate-vaccination-events branch from f2310f6 to f3429dc Compare September 5, 2025 16:58
@thomasleese thomasleese added the operations Improving live support label Sep 5, 2025
@thomasleese thomasleese added this to the v4.1.0 milestone Sep 5, 2025
Copy link

sonarqubecloud bot commented Sep 5, 2025

@thomasleese thomasleese merged commit f1f2f76 into next Sep 5, 2025
13 checks passed
@thomasleese thomasleese deleted the add-method-and-cli-task-to-generate-vaccination-events branch September 5, 2025 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
operations Improving live support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants