Closed
Conversation
murugapl
force-pushed
the
new-patient-session-designs
branch
from
9fd2d26 to
14b8775
Compare
murugapl
force-pushed
the
new-patient-session-designs
branch
from
14b8775 to
81448fe
Compare
paulrobertlloyd
requested changes
Apr 13, 2026
Contributor
paulrobertlloyd
left a comment
paulrobertlloyd
left a comment
There was a problem hiding this comment.
First pass; can I review it again once we have the sticky header?
| end | ||
|
|
||
| def action_link_text | ||
| "View child's #{programme.name} record" |
Contributor
There was a problem hiding this comment.
Suggested change
| "View child's #{programme.name} record" | |
| "View child’s #{programme.name} record" |
| "#{patient.given_name} was vaccinated on #{record&.performed_at&.to_fs(:long)}." | ||
| end | ||
| when "needs_triage" | ||
| "You need to decide if it's safe to vaccinate." |
Contributor
There was a problem hiding this comment.
Suggested change
| "You need to decide if it's safe to vaccinate." | |
| "You need to decide if it’s safe to vaccinate." |
| <% end %> | ||
| <% elsif latest_triage.nil? || latest_triage.keep_in_triage? %> | ||
| <% if helpers.policy(Triage).new? %> | ||
| <p>You need to decide if <%= patient.full_name %> is safe to vaccinate.</p> |
Contributor
There was a problem hiding this comment.
Suggested change
| <p>You need to decide if <%= patient.full_name %> is safe to vaccinate.</p> | |
| <p>You need to decide if <%= patient.given_name %> is safe to vaccinate.</p> |
The national reporting teams uses the same vaccination table but requires more details on vaccination records like location and source, whereas full-fat Mavis doesn't need so much information. The new `show_details` arg controls whether to show or hide this info.
Shows the programme status (heading with colour, contextual detail text, vaccination outcomes table, and action link) for a patient on the session patient show page. This card links to the respective programme tab in the child’s record. Eventually this will be the only card on the page with a coloured heading so various statuses (consent, triage, etc) do not compete for attention and the programme status for the child is reflected right at the top. MAV-4846
We no longer show the coloured heading with consent/triage status, and we now use the new AppActionLinkComponent instaed of buttons for actions like recording a new response
This component is now broken out from the consent card and is shown as a standalone card on the patient session page. Slight updates to wording, and uses new AppActionLinkComponent for actions.
Shows a PSD status card ("PSD added" / "PSD not added") on the patient session page, only rendered when the session has PSD enabled.
Shows a registration card on the patient session page for sessions that require registration. Displays a form to mark the child as attending or absent when not yet registered, a status summary with an update link when already registered, and a completion message when the session is done
Make the patient session banner sticky as the user scrolls and use the same design as for session pages. The banner should collapse showing minimal information as the user scrolls
murugapl
force-pushed
the
new-patient-session-designs
branch
from
81448fe to
92d108c
Compare
Contributor
Author
It should be sticky I think! You may need to restart server as it was a new stylesheet |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Jira Issue - MAV-4846
Screenshots
...