Patient detail changes for new scenarios - #835
Merged
Merged
Conversation
Co-Authored-By: Anna-Sutton <184676072+Anna-Sutton@users.noreply.github.com>
Co-Authored-By: Anna-Sutton <184676072+Anna-Sutton@users.noreply.github.com>
frankieroberto
temporarily deployed
to
record-a-vac-patient-de-yxuqb7
August 10, 2026 15:12
Inactive
frankieroberto
temporarily deployed
to
record-a-vac-patient-de-yxuqb7
August 11, 2026 09:39
Inactive
frankieroberto
temporarily deployed
to
record-a-vac-patient-de-yxuqb7
August 12, 2026 10:33
Inactive
benjystanton
marked this pull request as ready for review
August 12, 2026 12:48
caitlinroach-nhs
left a comment
Contributor
There was a problem hiding this comment.
Just a couple of suggestions that we may want to add to the prototype:
- It might be worth creating a new user preset for a GP: I know the actual flow overall doesn't change much, but if it does in the future, it would be nice to have that there already so we can easily update. (presets being the list of user scenarios on the index page).
- It might be worth creating a quick link to the page with the warning that occurs when there is no NHS number (can just be a static page) just to make it a bit easier for devs and Ileri to locate when needed for reference
frankieroberto
temporarily deployed
to
record-a-vac-patient-de-yxuqb7
August 12, 2026 14:27
Inactive
frankieroberto
temporarily deployed
to
record-a-vac-patient-de-yxuqb7
August 13, 2026 07:14
Inactive
Contributor
Author
|
@caitlinroach-nhs good ideas, thanks. I've done those now. The GP user preset needs refinement. But can tweak that as I go I think. I've also added some direct links to the new covid warning messages further down. |
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
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.
Add in some content warnings to some GP scenarios, so that users are informed about any issues that may impact payment reconciliation.
Page changes
indexAdded quick links to GP scenariosrecord-vaccinations/patient-historyAdded conditional inset text and warning calloutrecord-vaccinations/patient-history-no-gprecord-vaccinations/patient-history-noneChanged next page, made some content dynamic, added conditional warning call outrecord-vaccinations/covid-19/not-in-pcn-warningrecord-vaccinations/covid-19/no-gp-surgeryData and route changes
app/data/organisations.jsAdd a fake GP surgeryapp/data/user.jsAdd an example GP user linked to GP orgapp/routes/prototype-admin.jsCreate a quick access route and preset for new GP user and orgapp/routes/record-vaccinations.jsAdd routes for GP scenariosScenario quick links
Also added some quick access links to the index for GP scenarios, including query strings to help show correct content.