Skip to content

[PRMP-775] Implement download success page #1007

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
Merged
adamwhitingnhs merged 2 commits into from
Jan 30, 2026
Merged

[PRMP-775] Implement download success page #1007

adamwhitingnhs merged 2 commits into from
Jan 30, 2026

Conversation

@adamwhitingnhs
Copy link
Contributor

@adamwhitingnhs adamwhitingnhs commented Jan 21, 2026 •
edited
Loading

Overview

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

Description

Add download success page and navigate to it when user downloads a record.

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.

download-success

@adamwhitingnhs adamwhitingnhs marked this pull request as ready for review January 21, 2026 11:58
@adamwhitingnhs adamwhitingnhs requested review from a team as code owners January 21, 2026 11:58
lillie-dae
lillie-dae previously approved these changes Jan 23, 2026
View reviewed changes
@adamwhitingnhs adamwhitingnhs force-pushed the PRMP-775 branch 2 times, most recently from 237b28b to 38dd1c8 Compare January 28, 2026 09:34
@github-actions
Copy link

github-actions bot commented Jan 28, 2026

Code security issues found

View full details here.

@adamwhitingnhs adamwhitingnhs enabled auto-merge (squash) January 28, 2026 17:36
@github-actions
Copy link

github-actions bot commented Jan 28, 2026

Code security issues found

View full details here.

@github-actions
Copy link

github-actions bot commented Jan 30, 2026

Code security issues found

View full details here.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 30, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

@adamwhitingnhs adamwhitingnhs merged commit 5c2d3ce into main Jan 30, 2026
17 of 18 checks passed
@adamwhitingnhs adamwhitingnhs deleted the PRMP-775 branch January 30, 2026 11:12
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 approved these changes

@kamenbachvarov-nhs kamenbachvarov-nhs kamenbachvarov-nhs approved these changes

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

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@adamwhitingnhs @lillie-dae @kamenbachvarov-nhs @DuncanSangsterNHS