Skip to content

PO-8912-remove-back-button#2837

Open
marcmollins wants to merge 5 commits into
masterfrom
PO-8912-remove-back-button
Open

PO-8912-remove-back-button#2837
marcmollins wants to merge 5 commits into
masterfrom
PO-8912-remove-back-button

Conversation

@marcmollins

@marcmollins marcmollins commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

removing back button that should not be there

Jira link

https://tools.hmcts.net/jira/browse/PO-8912

Change description

removed a back button from the manual account creation screen that was agreed should not be there from approved designs. I have also updated unit tests.

Testing done

Unit testing and viewed the changes on local server

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

removing back button that should not be there

@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

@iamfrankiemoran iamfrankiemoran requested a review from a team July 9, 2026 16:08

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

fixing now broken functional tests

@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