Skip to content

Conversation

@swebberuk
Copy link
Contributor

@swebberuk swebberuk commented Nov 25, 2025

Description

Add update form for implanted medical device history item

image

Jira link

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

Review notes

I've implemented this by following the approach that was done in #760.

#771 has abstracted the pattern of adding a counter to the visually hidden text in the "change" link. I can refactor my changes to use that approach once it has been merged.

Review checklist

  • Check database queries are correctly scoped to current_provider

@swebberuk swebberuk force-pushed the DTOSS-11528-update-implanted-medical-device branch from 577567c to 62cd91c Compare November 25, 2025 16:08
@swebberuk swebberuk changed the title Dtoss 11528 update implanted medical device [wip] Add update form for implanted medical device history item Nov 25, 2025
@swebberuk swebberuk force-pushed the DTOSS-11528-update-implanted-medical-device branch from 62cd91c to b3d4d95 Compare November 25, 2025 16:26
@swebberuk swebberuk changed the title [wip] Add update form for implanted medical device history item Add update form for implanted medical device history item Nov 26, 2025
Copy link
Contributor

@MatMoore MatMoore 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. There's a few other bits of cleanup I'm planning to do due to naming inconsistencies and I think the headings/page titles are not all following the same pattern. But I'm going to wait until we've merged in all these forms.

@swebberuk swebberuk marked this pull request as ready for review November 26, 2025 14:20
@swebberuk swebberuk requested a review from a team November 26, 2025 14:20
@swebberuk swebberuk merged commit 18d414b into main Nov 26, 2025
12 checks passed
@swebberuk swebberuk deleted the DTOSS-11528-update-implanted-medical-device branch November 26, 2025 14:20
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