Skip to content

[PRMP-1377] Enable RUM and custom event#1035

Merged
adamwhitingnhs merged 3 commits intomainfrom
PRMP-1377
Feb 3, 2026
Merged

[PRMP-1377] Enable RUM and custom event#1035
adamwhitingnhs merged 3 commits intomainfrom
PRMP-1377

Conversation

@adamwhitingnhs
Copy link
Copy Markdown
Contributor

@adamwhitingnhs adamwhitingnhs commented Jan 30, 2026

Overview

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

Description

Enabled RUM in all environments and adds a custom event for when the user clicks the PDF preview print button.

Checklist

Tasks for all changes:

  • [x ] 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:

@adamwhitingnhs adamwhitingnhs requested review from a team as code owners January 30, 2026 16:27
SWhyteAnswer
SWhyteAnswer reviewed Feb 2, 2026
View reviewed changes
Comment thread app/src/components/generic/pdfViewer/PdfViewer.tsx
SWhyteAnswer
SWhyteAnswer reviewed Feb 2, 2026
View reviewed changes
Comment thread app/src/components/generic/pdfViewer/PdfViewer.tsx
SWhyteAnswer
SWhyteAnswer reviewed Feb 2, 2026
View reviewed changes
Comment thread app/src/components/layout/footer/Footer.tsx
SWhyteAnswer
SWhyteAnswer reviewed Feb 2, 2026
View reviewed changes
Comment thread app/src/components/blocks/_cookiesPolicy/cookiesPolicy/CookiesPolicy.tsx
SWhyteAnswer
SWhyteAnswer reviewed Feb 2, 2026
View reviewed changes
Comment thread app/src/components/blocks/_cookiesPolicy/cookiesPolicy/CookiesPolicy.tsx Outdated
SWhyteAnswer
SWhyteAnswer reviewed Feb 2, 2026
View reviewed changes
Comment thread app/src/components/generic/pdfViewer/PdfViewer.tsx Outdated
SWhyteAnswer
SWhyteAnswer reviewed Feb 2, 2026
View reviewed changes
Comment thread app/src/components/layout/Layout.tsx Outdated
SWhyteAnswer
SWhyteAnswer previously approved these changes Feb 2, 2026
View reviewed changes
@adamwhitingnhs adamwhitingnhs enabled auto-merge (squash) February 3, 2026 10:13
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 3, 2026

Code security issues found

View full details here.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 3, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
1 Accepted issue

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

See analysis details on SonarQube Cloud

@adamwhitingnhs adamwhitingnhs merged commit 3474808 into main Feb 3, 2026
25 of 26 checks passed
@adamwhitingnhs adamwhitingnhs deleted the PRMP-1377 branch February 3, 2026 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@robg-test robg-test robg-test approved these changes

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

+2 more reviewers

@SWhyteAnswer SWhyteAnswer SWhyteAnswer approved these changes

@oliverbeumkes-nhs oliverbeumkes-nhs oliverbeumkes-nhs left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@adamwhitingnhs @robg-test @SWhyteAnswer @oliverbeumkes-nhs @NogaNHS