Skip to content

Appointments MVP (MYA integration)#762

Open
caitlinroach-nhs wants to merge 37 commits into
mainfrom
appointments-sketch-5
Open

Appointments MVP (MYA integration)#762
caitlinroach-nhs wants to merge 37 commits into
mainfrom
appointments-sketch-5

Conversation

@caitlinroach-nhs
Copy link
Copy Markdown
Contributor

This is work for the MYA integration into RAVS to show appointments.

It adds a new page called Appointments and contains:

  • List of appointments for Yesterday, Today and Tomorrow
  • Ability to record vaccination against an appointment on Today

frankieroberto and others added 27 commits January 23, 2026 15:00
Changes include:
- Remove batch and vaccinator selection
- Add nhs number in tables
- add day of week alongside date
- remove cancel option
- Add cancellation reason
- Add same vaccs, diff patient option when coming from appt
- Only show yesterday, today and tomorrow and no actions on yest./tom.
- change complete table to match columns from scheduled (except actions)
-Time format fix
@frankieroberto frankieroberto temporarily deployed to record-a-vac-appointmen-arbv60 May 7, 2026 16:02 Inactive
@caitlinroach-nhs caitlinroach-nhs marked this pull request as ready for review May 13, 2026 16:09
@caitlinroach-nhs caitlinroach-nhs requested review from Anna-Sutton and frankieroberto and removed request for Anna-Sutton May 13, 2026 16:10
Copy link
Copy Markdown
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.

Looks good! Some minor comments.

Comment thread app/views/appointments/_cancelled.html Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This table now looks pretty tight with 6 columns! Any thoughts on how we might give it a bit more breathing space? 🤔

Image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I agree -- I've added to the questions for Mark for the remaining sessions to see if all of this information is still useful in the canceled table. But will also give alternatives some thought

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do you know if these are the reasons that would be given for the cancellation? Looking at MYA but they don't show reasons in the tables of cancelled appointments

Copy link
Copy Markdown
Contributor Author

@caitlinroach-nhs caitlinroach-nhs May 18, 2026

Choose a reason for hiding this comment

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

Some options for more space @frankieroberto -- widen the content area past 990 across the board (MYA's is 1099) or make part of the table scroll horizontally (not sure what the POV is on scrollable tables in NHS? Only reason I suggest it here is that it would be minimal and wouldn't hide any information or actions completely).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@caitlinroach-nhs we were told that there are 3 ways an appointment can be cancelled:

  1. the patient themselves cancels (for some reason - not sure if they specify why?)
  2. the pharmacy cancels it (again, not sure if a reason is specified!)
  3. 'automatic' - which I believe happens if a system detects that they’ve had the vaccination elsewhere (eg as a walk-in).

Worth checking though!

Assuming those are the 3 reasons and we don't get any further details, we can still choose which words best represent those 3 possibilities.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@caitlinroach-nhs on the content width, we could make the whole service wider than the default 960px - that might help on some other screens too. There’s some guidance on that here: https://service-manual.nhs.uk/design-system/styles/layout

The only question is, how confident can we be that pharmacies generally have wider browser windows than 960px? I've seen some pretty small screens and laptops on the pharmacy counters and in consultation rooms! What was it like in your visit last week?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We saw all kinds -- most wouldn't have had an issue with a slightly larger width, with the exception of one that was super short and wide and unclear how they actually view anything on it. And for those who have had MYA and RAVS up side-by-side, having the appointments in RAVS eliminates that need, which would buy space back

Comment on lines +42 to +48
{% if appointment.cancelled.reason == "patient" %}
Patient
{% elif appointment.cancelled.reason == "organisation" %}
Us
{% elif appointment.cancelled.reason == "auto" %}
Automatic
{% endif %}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Do these make sense withe the 'Cancellation reason' heading, or does it need a bit more explanation? 🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The very brief feedback from the pharmacy visits indicated that they made sense, but also sent this question to Mark for the remaining sessions to see if we can get a little more info on this.

Comment thread app/views/appointments/_cancelled.html Outdated
Comment thread app/views/appointments/_completed.html
Comment thread app/views/appointments/_scheduled.html Outdated
Comment thread app/views/appointments/_scheduled.html Outdated
Co-authored-by: Frankie Roberto <frankie.roberto1@nhs.net>
@frankieroberto frankieroberto temporarily deployed to record-a-vac-appointmen-arbv60 May 18, 2026 13:44 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-appointmen-arbv60 May 18, 2026 13:45 Inactive
caitlinroach-nhs and others added 2 commits May 18, 2026 14:46
Co-authored-by: Frankie Roberto <frankie.roberto1@nhs.net>
@frankieroberto frankieroberto temporarily deployed to record-a-vac-appointmen-arbv60 May 18, 2026 13:47 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-appointmen-arbv60 May 18, 2026 13:48 Inactive
Fixed Lorna Biscuitbarrel's nhs number - was one digit too long so wasnt' formatting properly
@frankieroberto frankieroberto temporarily deployed to record-a-vac-appointmen-arbv60 May 18, 2026 13:51 Inactive
@frankieroberto frankieroberto temporarily deployed to record-a-vac-appointmen-arbv60 May 18, 2026 14:02 Inactive
Age was hard coded on the patient details page -- have updated to be generated based on the birthdate
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