Skip to content

PO-2391 - #2890

Open
louisbriggs wants to merge 13 commits into
PO-2389from
PO-2391
Open

PO-2391#2890
louisbriggs wants to merge 13 commits into
PO-2389from
PO-2391

Conversation

@louisbriggs

Copy link
Copy Markdown
Contributor

Jira link

See PO-2391

Change description

Added consolidated accounts tab contents to defendant accounts.
Added table to tab to display all consolidated accounts.

Testing done

Verified that the endpoint returns the correct data for the consolidated accounts.
Verified that the table contents are rendered correctly based on mock data.

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 self-assigned this Jul 22, 2026
@louisbriggs
louisbriggs marked this pull request as ready for review July 22, 2026 09:24
@louisbriggs
louisbriggs requested a review from a team July 22, 2026 09:24

@Arnabsubedi233 Arnabsubedi233 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@iamfrankiemoran iamfrankiemoran left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Minor changes

@iamfrankiemoran iamfrankiemoran left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM ✅

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.

5 participants