Skip to content

Conversation

@MatMoore
Copy link
Contributor

@MatMoore MatMoore commented Dec 8, 2025

Description

In the process of switching to small buttons, I noticed the mammogram history has not been implemented yet on the record medical history page.

This PR adds it in a minimal form, and wires up the add button. This aligns it with what we are showing on the view appointment page.

Screenshot of the new section, with a small 'Add another mammgram' button

Jira link

https://nhsd-jira.digital.nhs.uk/browse/DTOSS-11583

Review notes

We will need additional work to bring this card in line with the prototype, as illustrated in the screenshot below:

Screenshot showing the prototype version, which differs significantly
  1. We haven't implemented change or delete functionality for previous mammograms (DTOSS-11816)
  2. The subtitle references the "last confirmed" mammogram but this concept does not exist yet (DTOSS-11817)
  3. Attribution hasn't been implemented yet
  4. Review functionality and expander are both out of scope at the moment - this is fine I think

The highest priority are 1 & 2, which I've raised separate tickets for.

Review checklist

  • Check database queries are correctly scoped to current_provider

This is not yet complete, and it only shows reported mammograms, not
mammograms taken from the screening record.

I'm just making it consistent with what's already shown on the view
appointment page, under participant details.

The copy is slightly misleading because the last confirmed mammogram
isn't shown yet.

The "Add another mammogram" button on the record medical
information page is a small button, and submitting the form
should redirect back to the record medical information page.

In the prototype, the form redirects back to the specific section,
and highlights it with a border. We don't have this yet, but we can
implement it consistently for all sections in another ticket.
@MatMoore MatMoore marked this pull request as ready for review December 8, 2025 11:59
@MatMoore MatMoore requested a review from a team as a code owner December 8, 2025 11:59
@MatMoore MatMoore merged commit 07ce75e into main Dec 10, 2025
12 checks passed
@MatMoore MatMoore deleted the DTOSS-11583-mammograms branch December 10, 2025 11:34
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