Skip to content

Conversation

@cameronhargreaves1-nhs
Copy link
Contributor

@cameronhargreaves1-nhs cameronhargreaves1-nhs commented Dec 30, 2025

Description

Some clinic show page /clinic/ minor fixes:

User attribution in the appointment column should be grey (same styling as DOB age)
If the user is the current user the attribution name should be followed by "(you)"
NHS number should also be grey
"Go back" link should be "Clinics"

image image

Jira link

[DTOSS-11867](url)

Review notes

Review checklist

  • Check database queries are correctly scoped to current_provider

@cameronhargreaves1-nhs cameronhargreaves1-nhs requested a review from a team as a code owner December 30, 2025 15:01
return " (you)"
else:
return ""

Copy link
Contributor

@malcolmbaig malcolmbaig Jan 5, 2026

Choose a reason for hiding this comment

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

Nice — let's add tests for this behaviour too. A view test would be a sensible starting point, but in this particular case I think it's sufficient to update the status attribution system test to cover this change. The current user could be set as one of the attributed users, and then we can assert the presence of "(you)" in that attribution.

@cameronhargreaves1-nhs cameronhargreaves1-nhs changed the title [DTOSS-11867] Clinic show page snagging updates Clinic show page snagging updates Jan 5, 2026
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