Skip to content

Conversation

thomasleese
Copy link
Contributor

This adds a new tab to the campaigns page allowing users to see the cohort of patients associated with the campaign.

Screenshots

Screenshot 2024-09-05 at 16 10 02

@thomasleese thomasleese marked this pull request as ready for review September 5, 2024 16:11
@thomasleese thomasleese requested a review from a team as a code owner September 5, 2024 16:11
@thomasleese thomasleese force-pushed the refactor-patient-session-factories branch from e5eab23 to 2085aea Compare September 6, 2024 07:28
Base automatically changed from refactor-patient-session-factories to main September 6, 2024 07:32
This is through the sessions and patient sessions, although in the
future we might link patients with campaigns directly.
This adds a new tab for the campaigns which will be used to display the
cohort (patients) associated with a campaign.
To AppSessionPatientTableComponent, since it's used specifically when
managing sessions, and we'd like to add a new component that displays
just information on the patients.
This adds a new scope on patients for determining if they should be
considered "active", which in this case means they belong to a patient
session that itself is active. This is useful to prevent showing
patients in a campaign that haven't been fully imported yet.
This adds a component that we will use to display the cohort on the
campaign pages, showing a table of patients.
This updates the campaign patients tab to use the
`AppPatientTableComponent` to render a list of patients attached to the
campaign.
@thomasleese thomasleese enabled auto-merge (rebase) September 6, 2024 07:33
@tvararu tvararu temporarily deployed to mavis-pr-1710 September 6, 2024 07:33 Inactive
Copy link

sonarqubecloud bot commented Sep 6, 2024

@thomasleese thomasleese merged commit e6765dd into main Sep 6, 2024
11 checks passed
@thomasleese thomasleese deleted the cohort-tab branch September 6, 2024 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants