Show Gillick assessments in activity log #2519
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
thomasleese
force-pushed
the
gillick-assessments-activity-log
branch
from
cf9af90 to
5c0f836
Compare
thomasleese
force-pushed
the
gillick-assessments-activity-log
branch
from
5c0f836 to
41a9bc5
Compare
tvararu
approved these changes
Nov 20, 2024
This allows us to have multiple Gillick assessments per patient session, which means we can track when nurses change Gillick assessments by recording a new record each time.
This better represents their real database structure where the foreign key exists on the Gillick assessment model. It'll also allow us to modify the editing process to be amend-only, meaning that old Gillick assessments are kept around but only the latest one is shown to nurses. This makes it possible to show all Gillick assessments in the activity log for a patient.
When viewing the activity log for a patient we want to see all Gillick assessments that have been carried out, including any updates and changes.
This changes how Gillick assessments are editing by always creating a new record based on the previous one rather than updating the record. This allows us to keep a history of Gillick assessments per patient, and show them all in the activity log.
Because we now don't really have the concept of editing a Gillick assessment (they're always created new) then we can reduce duplication and go down to one set of controller actions.
thomasleese
force-pushed
the
gillick-assessments-activity-log
branch
from
41a9bc5 to
3c80b47
Compare
|
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 includes Gillick assessments in the activity log for a patient for better visibility. To support this, I've changed the behaviour of editing Gillick assessments to create new records based on the previous one which allows us to keep track of all Gillick assessments and show them all in the activity log.
Screenshot