Skip to content

CIV-9332 remove wording #3617

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 25 commits into
base: master
Choose a base branch
from
Open

CIV-9332 remove wording #3617

wants to merge 25 commits into from

Conversation

@MMNycz
Copy link
Contributor

@MMNycz MMNycz commented Aug 16, 2023

JIRA link

https://tools.hmcts.net/jira/browse/CIV-9332

Change description

remove wording

Work checklist

  • Unit tests added where applicable
  • Route tests added for new pages
  • New pages included in a11y tests
  • Task list and task completeness checks updated
  • Check and send page updated
  • Routes, page content and page flows of new features are toggled
  • UI changes look good on mobile
  • Required Google Analytics events are being sent

Developer self-QA run statement

  • I have clicked through the running application to see if all changes I made actually work.

Does this PR introduce a breaking change?

  • Yes
  • No

@MMNycz MMNycz removed the enableHelm label Aug 17, 2023
zikrurR
zikrurR approved these changes Aug 17, 2023
View reviewed changes
Copy link
Contributor

@zikrurR zikrurR left a comment

Choose a reason for hiding this comment

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

lgtm

sabahirfan
sabahirfan previously approved these changes Aug 17, 2023
View reviewed changes
@hmcts-jenkins-a-to-c hmcts-jenkins-a-to-c bot requested a deployment to preview September 1, 2023 05:50 Abandoned
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sabahirfan sabahirfan sabahirfan left review comments

+1 more reviewer

@zikrurR zikrurR zikrurR approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

ns:money-claims prd:cmc rel:cmc-citizen-frontend-pr-3617

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MMNycz @sabahirfan @zikrurR