Skip to content

Conversation

@NogaNHS
Copy link
Contributor

@NogaNHS NogaNHS commented Jan 19, 2026

Overview

Jira ticket: PRMP-1227

Description

document uploads that fail demographic or business validation to be automatically sent to the Review Queue

Checklist

Tasks for all changes:

@NogaNHS NogaNHS changed the base branch from PRMP-1218 to main January 19, 2026 16:27
@NogaNHS NogaNHS changed the base branch from main to PRMP-1218 January 19, 2026 16:27
@steph-torres-nhs steph-torres-nhs changed the base branch from PRMP-1218 to main January 19, 2026 17:01
@steph-torres-nhs steph-torres-nhs changed the base branch from main to PRMP-1218 January 19, 2026 17:01
@NogaNHS NogaNHS changed the base branch from PRMP-1218 to main January 19, 2026 17:04
@adamwhitingnhs adamwhitingnhs changed the base branch from main to PRMP-1218 January 20, 2026 14:26
self.dynamo_repository.write_report_upload_to_dynamo(
failed_entry, UploadStatus.FAILED, str(error)
)
failed_files[(nhs_number, ods_code)].append(failed_file)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does order matter here? would we still want to append to failed files even if writing to dynamo for reporting fails
do we handle this failure?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants