Skip to content

Conversation

thomasleese
Copy link
Contributor

This renames the draft column on Sessions to active, to be consistent with the Campaign and PatientSession model, which then allows us to introduce a Draftable concern (not entirely sure on the name, but it was consistent with Recordable) that encapsulates common logic.

Base automatically changed from draft-patient-sessions to main September 5, 2024 14:02
This will be renamed to active to match the column naming convention we
have in the other tables, to improve consistency.
This updates the code in a number of places to support the rename of the
session draft column from `draft` to `active`, the functionality should
remain the same before and after.
This adds a new concern to encapsulate the logic around models which
have an active field, and two states of either draft or active.
Copy link

sonarqubecloud bot commented Sep 5, 2024

@thomasleese thomasleese marked this pull request as ready for review September 5, 2024 14:03
@thomasleese thomasleese requested a review from a team as a code owner September 5, 2024 14:03
@tvararu tvararu temporarily deployed to mavis-pr-1708 September 5, 2024 14:05 Inactive
@tvararu tvararu merged commit 8c5c778 into main Sep 6, 2024
11 checks passed
@tvararu tvararu deleted the draftable-concern branch September 6, 2024 07:21
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