Skip to content

PO-2673 changes - #2879

Open
bpires2 wants to merge 37 commits into
masterfrom
PO-2673
Open

PO-2673 changes#2879
bpires2 wants to merge 37 commits into
masterfrom
PO-2673

Conversation

@bpires2

@bpires2 bpires2 commented Jul 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Jira link

See PO-2673

Change description

  • Changes to make defendant, minor and major account creditor screens responsive up to 400%

Testing done

N/A

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

@bpires2
bpires2 requested a review from a team July 20, 2026 14:28
iamfrankiemoran
iamfrankiemoran requested changes Jul 20, 2026
View reviewed 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.

Minor change

Comment thread src/app/flows/fines/fines-acc/fines-acc-summary-header/fines-acc-summary-header.component.html
iamfrankiemoran
iamfrankiemoran approved these changes Jul 21, 2026
View reviewed 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 ✅

louisbriggs
louisbriggs approved these changes Jul 21, 2026
View reviewed changes

@louisbriggs louisbriggs 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

@hmcts-jenkins-cnp
hmcts-jenkins-cnp Bot requested a deployment to preview July 28, 2026 12:28 Abandoned
iamfrankiemoran
iamfrankiemoran approved these changes Jul 30, 2026
View reviewed 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

Reviewers

@iamfrankiemoran iamfrankiemoran iamfrankiemoran approved these changes
@louisbriggs louisbriggs louisbriggs approved these changes

Assignees

@bpires2 bpires2

Labels

ns:opal prd:opal rel:opal-frontend-pr-2879

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@bpires2 @iamfrankiemoran @louisbriggs @benji-telagalapudi76 @Ju-Morgan