feature/PIN-8797-add-dowload-risk-analysis#1611
Conversation
martinaCampoli
requested review from
Alepazz,
Carminepo2,
borgesis95 and
ruggerocastagnola
as code owners
There was a problem hiding this comment.
Pull request overview
This PR implements the download functionality for signed risk analysis documents in purpose templates, addressing ticket PIN-8797. The implementation adds a new download hook and service method, updates localization strings in both Italian and English, and enables the download button in the UI component.
Changes:
- Added download capability for signed risk analysis documents in purpose templates
- Updated localization files with new error and loading messages
- Updated tests to verify the presence of the download button
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/static/locales/it/mutations-feedback.json | Added Italian translations for download risk analysis loading and error messages |
| src/static/locales/en/mutations-feedback.json | Added English translations for download risk analysis loading and error messages |
| src/pages/ConsumerPurposeTemplateCatalogDetailsPage/test/ConsumerPurposeTemplateCatalogDetailsTab.test.tsx | Updated test to verify download button is now rendered |
| src/components/shared/PurposeTemplate/PurposeTemplateDetailsTab/PurposeTemplateGeneralnfoSection.tsx | Implemented download button with handler using the new download hook |
| src/api/purposeTemplate/purposeTemplate.services.ts | Added downloadSignedRiskAnalysis service function to fetch signed risk analysis documents |
| src/api/purposeTemplate/purposeTemplate.downloads.ts | Added useDownloadSignedRiskAnalysis hook to handle download with user feedback |
| src/api/api.generatedTypes.ts | Updated generated API types including new endpoint definitions and minor documentation updates |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...onents/shared/PurposeTemplate/PurposeTemplateDetailsTab/PurposeTemplateGeneralnfoSection.tsx
Outdated
Show resolved
Hide resolved
src/api/purposeTemplate/__test__/purposeTemplate.services.test.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...onents/shared/PurposeTemplate/PurposeTemplateDetailsTab/PurposeTemplateGeneralnfoSection.tsx
Show resolved
Hide resolved
src/api/purposeTemplate/__test__/purposeTemplate.services.test.ts
Outdated
Show resolved
Hide resolved
src/api/purposeTemplate/__test__/purposeTemplate.services.test.ts
Outdated
Show resolved
Hide resolved
|
2 participants
Issue
PIN-8797
Description / Context / Why