Skip to content

PO-1866#2317

Open
louisbriggs wants to merge 49 commits intomasterfrom
PO-1866
Open

PO-1866#2317
louisbriggs wants to merge 49 commits intomasterfrom
PO-1866

Conversation

@louisbriggs
Copy link
Copy Markdown
Contributor

Jira link

See PO-1866

Change description

Adding Add/Change Enforcement Override page to account summary.

Testing done

Covered new components with unit test.

Security Vulnerability Assessment

CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@louisbriggs louisbriggs requested a review from a team March 11, 2026 09:29
@louisbriggs louisbriggs marked this pull request as ready for review March 11, 2026 09:30
Copy link
Copy Markdown
Contributor

@iamfrankiemoran iamfrankiemoran left a comment

Choose a reason for hiding this comment

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

Minor changes

Copy link
Copy Markdown
Contributor

@iamfrankiemoran iamfrankiemoran left a comment

Choose a reason for hiding this comment

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

Minor comment

Copy link
Copy Markdown
Contributor

@iamfrankiemoran iamfrankiemoran left a comment

Choose a reason for hiding this comment

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

LGTM

@louisbriggs louisbriggs requested a review from a team March 23, 2026 10:01
Copy link
Copy Markdown
Contributor

@Ju-Morgan Ju-Morgan left a comment

Choose a reason for hiding this comment

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

Confirmed test ACs covered and in line. Ready for merge.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants