Skip to content

Add resume appointment link#879

Merged
cameronhargreaves1-nhs merged 1 commit intomainfrom
DTOSS-11871-resume-appointment-link
Jan 19, 2026
Merged

Add resume appointment link#879
cameronhargreaves1-nhs merged 1 commit intomainfrom
DTOSS-11871-resume-appointment-link

Conversation

@cameronhargreaves1-nhs
Copy link
Copy Markdown
Contributor

@cameronhargreaves1-nhs cameronhargreaves1-nhs commented Jan 8, 2026

"Resume Appointment" link has been created. This link will be shown on an appointment that has been started by the current logged in user and has not been finished.

Currently the link will send the user to the confirm identity page, as the AppointmentStates have not been updated as the user goes through the workflow. There is to be a second PR after the Appointment States work is merged, to update the resume appointment link url with the furthest point in the workflow that hasn't been completed.

Description

Jira link

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

Review notes

Review checklist

  • Check database queries are correctly scoped to current_provider

@cameronhargreaves1-nhs cameronhargreaves1-nhs force-pushed the DTOSS-11871-resume-appointment-link branch 3 times, most recently from 9ffc9b6 to 7d565b3 Compare January 13, 2026 16:01
@cameronhargreaves1-nhs cameronhargreaves1-nhs marked this pull request as ready for review January 13, 2026 16:06
@cameronhargreaves1-nhs cameronhargreaves1-nhs requested a review from a team January 13, 2026 16:06
Comment thread manage_breast_screening/core/jinja2/components/resume-appointment/template.jinja Outdated
Comment thread manage_breast_screening/core/jinja2/components/resume-appointment/template.jinja Outdated
@cameronhargreaves1-nhs cameronhargreaves1-nhs force-pushed the DTOSS-11871-resume-appointment-link branch 3 times, most recently from 6fc454d to 5908240 Compare January 14, 2026 11:31
Comment thread manage_breast_screening/clinics/jinja2/clinics/show.jinja Outdated
@cameronhargreaves1-nhs cameronhargreaves1-nhs force-pushed the DTOSS-11871-resume-appointment-link branch from 5908240 to ad2a706 Compare January 14, 2026 11:40
Comment thread manage_breast_screening/core/jinja2/components/resume-appointment/template.jinja Outdated
return (
self._appointment.current_status.is_in_progress()
and user.pk == self._appointment.current_status.created_by.pk
)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just fyi we were discussing another edge case on slack yesterday, where the current user leaves an appointment in progress without pausing it https://screening-discovery.slack.com/archives/C082XV3GQVA/p1768304234462379

Think we are going to address this in another ticket though (cc @gpeng )

The link will only show for appointments the current user has started
Currently, the link will send the user to the confirm identity page
while the AppointmentStatus work is being worked on.
@cameronhargreaves1-nhs cameronhargreaves1-nhs force-pushed the DTOSS-11871-resume-appointment-link branch from ad2a706 to 30e3175 Compare January 16, 2026 10:48
@sonarqubecloud
Copy link
Copy Markdown

@cameronhargreaves1-nhs cameronhargreaves1-nhs merged commit c39695d into main Jan 19, 2026
14 checks passed
@cameronhargreaves1-nhs cameronhargreaves1-nhs deleted the DTOSS-11871-resume-appointment-link branch January 19, 2026 14:08
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.

3 participants