-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build new patient session page design #3169
Open
thomasleese
wants to merge
20
commits into
main
Choose a base branch
from
redesign-patient-session
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
4b79fc8
to
979564d
Compare
979564d
to
62fe355
Compare
62fe355
to
a1ff7be
Compare
a1ff7be
to
4da1abf
Compare
4da1abf
to
e79b665
Compare
e79b665
to
3e6d2eb
Compare
3e6d2eb
to
3699968
Compare
3699968
to
9cfc142
Compare
9cfc142
to
809f150
Compare
809f150
to
618e3cf
Compare
a4b86af
to
592ac27
Compare
592ac27
to
8f1b43d
Compare
This adds a new component that renders the patient summary information shown on the side of the patient session page containing the name, date of birth, address and a link to the full record.
This refactors the helper used to generate a status tag for a consent response to match the latest designs in the prototype.
This adds a component that renders the table of consent responses, which comes from the existing `AppPatientPageComponent`.
acd2ec2
to
5074ccc
Compare
This adds a component that renders the consent section of the patient session page, containing the consent card, gillick assessments and health answers.
This refactors the component to simplify its use and make it work in a similar way to the `AppVaccinateFormComponent`.
This component renders a list of triage notes and replaces the existing `AppTriageNotesComponent` to match the latest designs in the prototype.
06cfc8a
to
ff34e0a
Compare
This adds a component that renders the triage section of the patient session page, containing the consent card, gillick assessments and health answers.
From `AppSimpleStatusBannerComponent` as the link will show instead in the triage section.
This refactors the component to handle the creation of the default `VaccinateForm` object to simplify the `AppPatientPageComponent` in preparation for its removal.
This adds a component that renders the record section of the patient session page, containing the pre-screening questions and vaccinate form.
And move these methods in to the `VaccinesHelper`.
This adds the functionality to render the location of a vaccination record to be used in multiple places in the service.
This updates the component to match the latest designs in the prototype where the date, location, programme and outcome is shown.
This adds a component which renders the programme outcome section of the patient session page, which lists the vaccination records for the patient.
This is no longer being used as it's been replaced with separate patient session components. This component doesn't follow the latest designs in the prototype.
This is no longer being used so it can be safely removed.
It's no longer being used so it can be safely removed.
We no longer have a single status for patient sessions and this is no longer be used so it can be safely removed.
ff34e0a
to
49a18d8
Compare
|
tvararu
approved these changes
Mar 15, 2025
<% end %> | ||
|
||
<% if (notes = gillick_assessment.notes).present? %> | ||
<p class="nhsuk-body"><%= notes %></p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
class="nhsuk-body"
is a no-op in our app
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 builds the new designs for the patient session page to match the latest designs in the prototype.
Four new sections are added which shows on the patient session page:
This is a big PR and probably easier to review commit by commit. In theory we could extract the different parts in to separate PRs if that's helpful, although in general the page doesn't work as well in a half-redesigned state.
Screenshots