Skip to content

[PRMP-1476] Update User Restriction ODS Code upon MNS Notification (Active Records Only)#1164

Open
steph-torres-nhs wants to merge 4 commits intomainfrom
PRMP-1476
Open

[PRMP-1476] Update User Restriction ODS Code upon MNS Notification (Active Records Only)#1164
steph-torres-nhs wants to merge 4 commits intomainfrom
PRMP-1476

Conversation

@steph-torres-nhs
Copy link
Contributor

@steph-torres-nhs steph-torres-nhs commented Mar 10, 2026 •
edited
Loading

Overview

Jira ticket: PRMP-1476

Description

Add in an update to user restrictions on MNS event processing

Context

User Restrictions feature

Checklist

Tasks for all changes:

@github-actions
Copy link

github-actions bot commented Mar 20, 2026

Code security issues found

View full details here.

NogaNHS
NogaNHS reviewed Mar 20, 2026
View reviewed changes
lambdas/services/process_mns_message_service.py
logger.info(f"Updating restriction {restriction.id}")
self.restrictions_dynamo_service.update_restriction_custodian(
restriction_id=restriction.id,
updated_custodian=custodian,
Copy link
Contributor

@NogaNHS NogaNHS Mar 20, 2026

Choose a reason for hiding this comment

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

Last updated?

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 20, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
86.3% 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

@SteveMahoneyNHS SteveMahoneyNHS Awaiting requested review from SteveMahoneyNHS SteveMahoneyNHS 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.

2 participants

@steph-torres-nhs @NogaNHS