Skip to content

PO-5755 - #2880

Open
Arnabsubedi233 wants to merge 19 commits into
masterfrom
po-5755
Open

PO-5755#2880
Arnabsubedi233 wants to merge 19 commits into
masterfrom
po-5755

Conversation

@Arnabsubedi233

Copy link
Copy Markdown
Contributor

Jira link

See https://tools.hmcts.net/jira/browse/PO-5755

Change description

  • Added permanent banner on certain account statuses

Testing done

  • locally tested
  • unit tested

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

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

4 participants