Skip to content

Patient detail changes for new scenarios - #835

Merged
benjystanton merged 32 commits into
mainfrom
patient-detail-changes-for-new-scenarios
Aug 13, 2026
Merged

Patient detail changes for new scenarios#835
benjystanton merged 32 commits into
mainfrom
patient-detail-changes-for-new-scenarios

Conversation

@benjystanton

@benjystanton benjystanton commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Add in some content warnings to some GP scenarios, so that users are informed about any issues that may impact payment reconciliation.

Page changes

  • Updated index Added quick links to GP scenarios
  • Updated record-vaccinations/patient-history Added conditional inset text and warning callout
  • New record-vaccinations/patient-history-no-gp
  • Updated record-vaccinations/patient-history-none Changed next page, made some content dynamic, added conditional warning call out
  • New record-vaccinations/covid-19/not-in-pcn-warning
  • New record-vaccinations/covid-19/no-gp-surgery

Data and route changes

  • Updated app/data/organisations.js Add a fake GP surgery
  • Updatedapp/data/user.js Add an example GP user linked to GP org
  • Updated app/routes/prototype-admin.js Create a quick access route and preset for new GP user and org
  • Updated app/routes/record-vaccinations.js Add routes for GP scenarios

Scenario quick links

Also added some quick access links to the index for GP scenarios, including query strings to help show correct content.

  • Record vaccinations (in PCN)
    • Simple journey that shows new inset text on patient history page, no further changes
  • Record vaccinations (not in PCN)
    • Shows warning callout on patient history page, then later if user selects COVID-19 and not outreach it show an interruption panel
  • Record vaccinations (no GP surgery)
    1. If user enters an NHS number, it shows a warning callout on patient history page, then later if user selects COVID-19 and not outreach it show an interruption panel
    2. If user selects no NHS number, then selects continue with no NHS number, it shows a warning callout on patient history page, then later if user selects COVID-19 and not outreach it show an interruption panel
      • Note: do not enter all details on patient search page, otherwise prototype will go down "patient found' route

@frankieroberto
frankieroberto temporarily deployed to record-a-vac-patient-de-yxuqb7 August 6, 2026 15:36 Inactive
Co-Authored-By: Anna-Sutton <184676072+Anna-Sutton@users.noreply.github.com>
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-patient-de-yxuqb7 August 10, 2026 15:12 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-patient-de-yxuqb7 August 12, 2026 10:33 Inactive
@benjystanton
benjystanton marked this pull request as ready for review August 12, 2026 12:48

@caitlinroach-nhs caitlinroach-nhs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
frankieroberto temporarily deployed to record-a-vac-patient-de-yxuqb7 August 12, 2026 14:27 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-patient-de-yxuqb7 August 13, 2026 07:14 Inactive
@benjystanton

Copy link
Copy Markdown
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.

@benjystanton
benjystanton merged commit bd23eb9 into main Aug 13, 2026
1 check passed
@benjystanton
benjystanton deleted the patient-detail-changes-for-new-scenarios branch August 13, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants