-
Notifications
You must be signed in to change notification settings - Fork 1
[PRMP-915] download report feature to support review report #983
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.
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
base: main
Are you sure you want to change the base?
Conversation
| vi.restoreAllMocks(); | ||
| }); | ||
|
|
||
| it('should fetch the report url from the API and trigger link click', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should these be parameterised/test both report scenarios?
| </p> | ||
|
|
||
| <Table.Panel heading="Documents to review" className="reviews-page" allowFullScreen> | ||
| <div className="mb-3" style={{ textAlign: 'right' }}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We shouldn't be using inline styles.
you could use class "nhsuk-u-text-align-right"
… types and add tests for download link
…lter in test_ods_report_service.py
56d7594
Code security issues foundView full details here. |
|



Overview
Jira ticket: PRMP-915
Description
Refactored the download report functionality to support review report type. Added functionality for downloading the ODS review summary report on the
ReviewsPage. Includes relevant new tests.Checklist
Tasks for all changes:
I have run git pre-commits.(WIP)SANDBOX Full- Deploy feature branch to sandbox- workflow run - 20720089225Additional tasks for UI changes (delete if not applicable):