Skip to content

Error msgs for PDS and vax history issues#698

Merged
Anna-Sutton merged 19 commits intomainfrom
error-pages-for-vax-history-and-pds-issues
Mar 19, 2026
Merged

Error msgs for PDS and vax history issues#698
Anna-Sutton merged 19 commits intomainfrom
error-pages-for-vax-history-and-pds-issues

Conversation

@Anna-Sutton
Copy link
Contributor

@Anna-Sutton Anna-Sutton commented Mar 17, 2026

Amended some of the proposed error pages as follows:

If PDS lookup fails
image

If we cannot show any vaccination history
image

If we can only show some vaccination history
image

Changed name of file and added 'temporary' to the msg.
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 09:39 Inactive
…tory.html

Renamed file, added temporary to the error msg, and tried to make clarify what may or may not be shown to the user.
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 09:44 Inactive
Changed file name and added 'temporary' to the msg.
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 09:46 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 09:47 Inactive
@Anna-Sutton
Copy link
Contributor Author

Anna-Sutton commented Mar 17, 2026

@frankieroberto I've made some changes to these error msgs. Let's review together later.

Things to consider:

  • the H1s are quite long
  • not sure that the msg about incomplete vax history is clear
  • wondering if we should offer the option to Continue with no NHS number as a secondary button on the no patient details one.
  • have also changed the rate limit (429) error while I was looking at errors, but needs reviewing - not sure we even need a separate msg for this or if our 500 msg works for this scenario

Added link to the guidance page where you can download the paper form.
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 10:00 Inactive
Changed the msg so we're not making it sound like it's the user's fault.
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 10:05 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 10:07 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 10:08 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 14:13 Inactive
Added link to paper form.
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 14:36 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 14:47 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 14:49 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 15:03 Inactive
For now I've used the same msg as on our 500 error - pending conversation with devs about when we'd display this?
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 17, 2026 15:14 Inactive
@Anna-Sutton Anna-Sutton marked this pull request as ready for review March 17, 2026 15:15
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 18, 2026 11:03 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 18, 2026 11:04 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 18, 2026 11:06 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 18, 2026 11:11 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-error-page-0yhytu March 18, 2026 11:12 Inactive
@Anna-Sutton
Copy link
Contributor Author

@frankieroberto I have changed buttons to Nunjucks versions.

Copy link
Collaborator

@frankieroberto frankieroberto left a comment

Choose a reason for hiding this comment

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

Nice!

@Anna-Sutton Anna-Sutton merged commit 07b93c6 into main Mar 19, 2026
1 check passed
@Anna-Sutton Anna-Sutton deleted the error-pages-for-vax-history-and-pds-issues branch March 19, 2026 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants