Skip to content

Troubleshooting all the things#785

Merged
caitlinroach-nhs merged 8 commits into
mainfrom
troubleshooting-june2026
Jun 8, 2026
Merged

Troubleshooting all the things#785
caitlinroach-nhs merged 8 commits into
mainfrom
troubleshooting-june2026

Conversation

@caitlinroach-nhs

@caitlinroach-nhs caitlinroach-nhs commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Issues to look at:

  • Capitalisation of Flu is inconsistent
  • In some situations, the date and site questions are being skipped when they shouldn't be
  • The eligibility question is missing the the flow for flu (should come after dose question)
  • Some of the things that should only appear for print on appts are appearing in the UI

Removed a rule that was only applied to flu that was affecting the capitalisation. It should now follow the same rule as others like pertussis and shingles
@frankieroberto frankieroberto temporarily deployed to record-a-vac-troublesho-rhiyhv June 8, 2026 08:31 Inactive
@caitlinroach-nhs caitlinroach-nhs changed the title Flu capitalisation issues Troubleshooting all the things Jun 8, 2026
@caitlinroach-nhs

Copy link
Copy Markdown
Contributor Author

Capitalisation of flu should be fixed now unless there are places that is has been hard coded in that are incorrect. There was an extra rule being applied just to flu that was causing problems, but has now been removed

@caitlinroach-nhs

Copy link
Copy Markdown
Contributor Author

Eligibility question should be fixed. The logic was missing the question as it had been re-routed to the dose question first, but without then routing to the eligibility question. That has been fixed and double checked for others that also need both of these questions. Once the new dose q's come in, we'll need to double check this for any that have a dose q that didn't before

@caitlinroach-nhs

Copy link
Copy Markdown
Contributor Author

date and site questions seem to be fixed -- but could use another set of eyes to make sure

@frankieroberto frankieroberto temporarily deployed to record-a-vac-troublesho-rhiyhv June 8, 2026 09:24 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-troublesho-rhiyhv June 8, 2026 09:29 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-troublesho-rhiyhv June 8, 2026 09:31 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-troublesho-rhiyhv June 8, 2026 09:53 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-troublesho-rhiyhv June 8, 2026 10:19 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-troublesho-rhiyhv June 8, 2026 10:27 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-troublesho-rhiyhv June 8, 2026 10:30 Inactive
@caitlinroach-nhs caitlinroach-nhs marked this pull request as ready for review June 8, 2026 10:31
@caitlinroach-nhs caitlinroach-nhs merged commit 887b85c into main Jun 8, 2026
1 check passed
@caitlinroach-nhs caitlinroach-nhs deleted the troubleshooting-june2026 branch June 8, 2026 10:31
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