Skip to content

[PRMP-1057] Creation of Report Distribution Lambda#1026

Open
PedroSoaresNHS wants to merge 60 commits intomainfrom
PRMP-1057-2
Open

[PRMP-1057] Creation of Report Distribution Lambda#1026
PedroSoaresNHS wants to merge 60 commits intomainfrom
PRMP-1057-2

Conversation

@PedroSoaresNHS
Copy link
Contributor

@PedroSoaresNHS PedroSoaresNHS commented Jan 29, 2026

Overview

Jira ticket: (https://nhsd-jira.digital.nhs.uk/browse/PRMP-1057)

Description

Creation of Report Distribution Lambda

Context

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:

PedroSoaresNHS and others added 30 commits December 17, 2025 13:36
@PedroSoaresNHS
[PRMP-1054] Create report_orchestration
e61d3bd
@PedroSoaresNHS
[PRMP-1054] Updated variable name
3d9a291
@PedroSoaresNHS
[PRMP-1054] Updated method name
528efe0
@PedroSoaresNHS
[PRMP-1054] Created tests and formated code
7c55481
@PedroSoaresNHS
[PRMP-1054] Small updates to allow library to be imported and used
dfbff8d
@PedroSoaresNHS
[PRMP-1054] fixed unit tests
f69afbf
@PedroSoaresNHS
[PRMP-1054] added lambda layer to report_orchestration_lambda
326a6c8
@PedroSoaresNHS
Merge remote-tracking branch 'origin/main' into PRMP-1054
4c31c97 
# Conflicts:
#	.github/workflows/base-lambdas-reusable-deploy-all.yml
@PedroSoaresNHS
[PRMP-1054] fixed how the search works
31fc5e4
@PedroSoaresNHS
[PRMP-1054] fixed comment
d247394
@PedroSoaresNHS
Merge remote-tracking branch 'origin/main' into PRMP-1054
38ea089
@PedroSoaresNHS
[PRMP-1057] created distribution lambda
25fda59
@PedroSoaresNHS
Merge remote-tracking branch 'origin/main' into PRMP-1054
bdff697
@PedroSoaresNHS
Merge remote-tracking branch 'origin/PRMP-1054' into PRMP-1057
21b062c
@PedroSoaresNHS
[PRMP-1057] updated for the case it fails to access contacts table
4d9b466
@PedroSoaresNHS
[PRMP-1182] trying with step functions
4ebc15e
@PedroSoaresNHS
[PRMP-1182] fixed unit tests
5cecec9
@PedroSoaresNHS
[PRMP-1182] Improved query to dynamo, to use specific ranges
8cd2335
@PedroSoaresNHS
[PRMP-1057] adjusted time querie to dynamo
aa1ac5c
@PedroSoaresNHS
[PRMP-1057] added logging, removed 1 todo
5d2a089
@robg-test
Merge branch 'main' into PRMP-1054
e7cfb8c
@PedroSoaresNHS
[PRMP-1057] fixed how items are grabbed from contact report table
68f7d3b
@PedroSoaresNHS
[PRMP-1057] fixed tests
cb0fd89
@PedroSoaresNHS
Merge remote-tracking branch 'origin/main' into PRMP-1057
4ba0003 
# Conflicts:
#	lambdas/services/base/dynamo_service.py
#	lambdas/tests/unit/services/base/test_dynamo_service.py
@PedroSoaresNHS
Merge remote-tracking branch 'origin/PRMP-1054' into PRMP-1057
5c3fa49
@PedroSoaresNHS
[PRMP-1057] removed commented line
5035fed
@PedroSoaresNHS
[PRMP-1057] removed redundancy in error message
0264411
@PedroSoaresNHS
[PRMP-1057] fixed comments
8f74fe3
@PedroSoaresNHS
[PRMP-1057] updated tests
509f3e9
@PedroSoaresNHS
[PRMP-1057] updated lambda names
0cf7002
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Code security issues found

View full details here.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 6, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

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

Reviewers

@NogaNHS NogaNHS NogaNHS left review comments

@steph-torres-nhs steph-torres-nhs steph-torres-nhs left review comments

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

At least 2 approving reviews are required to merge this pull request.

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

@PedroSoaresNHS @NogaNHS @steph-torres-nhs @SWhyteAnswer @robg-test