Skip to content
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

CIV-9332 remove wording #3617

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

@zikrurR zikrurR zikrurR approved these changes

@sabahirfan sabahirfan sabahirfan left review comments

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.

None yet
3 participants
@MMNycz @sabahirfan @zikrurR