Skip to content

[PRMP-1244] Changes to document view page buttons #1066

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
adamwhitingnhs wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[PRMP-1244] Changes to document view page buttons #1066

adamwhitingnhs wants to merge 1 commit into from

Conversation

@adamwhitingnhs
Copy link
Contributor

@adamwhitingnhs adamwhitingnhs commented Feb 10, 2026 •
edited
Loading

Overview

https://nhsd-jira.digital.nhs.uk/browse/PRMP-1244

Checklist

Tasks for all changes:

  • 1. I have linked this PR to its Jira ticket.
  • 2. I have run git pre-commits. (WIP)
  • 3. I have added and/or updated relevant tests.
  • 4. I have updated relevant documentation.
  • 5. I have considered the cross-team impact (and have PR approval from both Core & Demographics if necessary).
  • 6. I have successfully deployed this change to a sandbox and witnessed unit and e2e tests passing:

Additional tasks for UI changes (delete if not applicable):

  • 1. I have run the UI Smoke Tests against the deployed sandbox and witnessed it passing:
  • 2. I have added evidence (to this PR) e.g. screenshots/gifs of all visual changes.
image

@adamwhitingnhs adamwhitingnhs requested review from a team as code owners February 10, 2026 13:21
lillie-dae
lillie-dae previously approved these changes Feb 10, 2026
View reviewed changes
@adamwhitingnhs adamwhitingnhs force-pushed the PRMP-1244 branch 2 times, most recently from c4d56cb to 92ba9f5 Compare February 10, 2026 14:13
lillie-dae
lillie-dae previously approved these changes Feb 10, 2026
View reviewed changes
SWhyteAnswer
SWhyteAnswer previously approved these changes Feb 10, 2026
View reviewed changes
@adamwhitingnhs adamwhitingnhs force-pushed the PRMP-1244 branch 3 times, most recently from 25de184 to 5fb07e2 Compare February 10, 2026 14:52
@github-actions
Copy link

github-actions bot commented Feb 10, 2026

Code security issues found

View full details here.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 10, 2026

Quality Gate Passed Quality Gate passed

Issues
1 New issue
6 Accepted issues

Measures
0 Security Hotspots
91.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lillie-dae lillie-dae lillie-dae left review comments

@SWhyteAnswer SWhyteAnswer SWhyteAnswer left review comments

@SteveMahoneyNHS SteveMahoneyNHS Awaiting requested review from SteveMahoneyNHS SteveMahoneyNHS is a code owner automatically assigned from NHSDigital/access-store

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@adamwhitingnhs @lillie-dae @SWhyteAnswer