Skip to content

Comments

Outbreak consent requests#6095

Open
misaka wants to merge 6 commits intonextfrom
outbreak-consent-requests
Open

Outbreak consent requests#6095
misaka wants to merge 6 commits intonextfrom
outbreak-consent-requests

Conversation

@misaka
Copy link
Collaborator

@misaka misaka commented Feb 24, 2026

Add ability to create an MMR(V) session that uses outbreak style consent requests, that is to say, consent requests that are more urgent in tone.

There is a branch with the same name in the testing repo with updated tests to match these changes.

MAV-2073

Screenshots

image image image

@misaka misaka requested a review from a team as a code owner February 24, 2026 15:45
@misaka misaka force-pushed the outbreak-consent-requests branch from 3896f73 to 4425b3f Compare February 24, 2026 15:46
@misaka misaka added this to the v6.21.0 milestone Feb 24, 2026
@misaka misaka added the feature New functionality label Feb 24, 2026
@misaka misaka force-pushed the outbreak-consent-requests branch from 4425b3f to e602b73 Compare February 24, 2026 15:59
Add support for sessions to support outbreaks.

Jira-Issue: MAV-2073
This allows users to specify that the session should use outbreak-style
notifications.

Jira-Issue: MAV-2073
When showing session details include consent request style if the session
includes the MMR programme.

Jira-Issue: MAV-2073
When a session is MMR and flagged as "outbreak", send consent requests specific
to outbreaks.

Jira-Issue: MAV-2073
This allows the code to check for sessions that support outbreak without having
to be mmr-specific. It should allow us to support outbreaks for different
programmes in the future.

Jira-Issue: MAV-2073
@misaka misaka force-pushed the outbreak-consent-requests branch from a49b8dc to 5afa2dd Compare February 25, 2026 08:31
class AddOutbreakToSessions < ActiveRecord::Migration[8.1]
def change
add_column :sessions, :outbreak, :boolean, default: false
Session.update_all(outbreak: false)
Copy link
Contributor

Choose a reason for hiding this comment

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

For the migration to be reversible you'll need to add this:

Suggested change
Session.update_all(outbreak: false)
reversible do |dir|
dir.up do
Session.update_all(outbreak: false)
end
end

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh interesting. I tried a rollback locally and it worked. I wonder if I missed something or has this changed?

I'll have a look once I'm in the ofc.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh interesting, I'm surprised it worked, I wonder what it did about the update_all line 🤔

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ya, I was a little surprised too, tbh. I'll have a closer look, I probably missed something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants