Skip to content

CIV-12657 updated govuk frontend, changed crown image #3652

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 6 commits into
base: master
Choose a base branch
from
Open

CIV-12657 updated govuk frontend, changed crown image #3652

wants to merge 6 commits into from

Conversation

@dtortolaV1
Copy link
Contributor

@dtortolaV1 dtortolaV1 commented Feb 19, 2024 •
edited
Loading

JIRA link

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

Change description

Updated gov uk version, changed to new crown logo

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

@dtortolaV1 dtortolaV1 requested a review from a team as a code owner February 19, 2024 09:13
velimd
velimd previously approved these changes Feb 19, 2024
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dtortolaV1 @zikrurR @velimd