Skip to content

Fix Security Vulnerabilities #3648

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

Fix Security Vulnerabilities #3648

wants to merge 13 commits into from

Conversation

@Andrew-McMahon7
Copy link
Contributor

@Andrew-McMahon7 Andrew-McMahon7 commented Feb 5, 2024

JIRA link

Change description

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

@Andrew-McMahon7 Andrew-McMahon7 requested a review from a team as a code owner February 5, 2024 10:32
@hmcts-jenkins-a-to-c hmcts-jenkins-a-to-c bot requested a deployment to preview February 5, 2024 10:48 Abandoned
@hmcts-jenkins-a-to-c hmcts-jenkins-a-to-c bot requested a deployment to preview February 5, 2024 11:13 Abandoned
@hmcts-jenkins-a-to-c hmcts-jenkins-a-to-c bot requested a deployment to preview February 5, 2024 11:38 Abandoned
@hmcts-jenkins-a-to-c hmcts-jenkins-a-to-c bot requested a deployment to preview February 5, 2024 11:56 Abandoned
@hmcts-jenkins-a-to-c hmcts-jenkins-a-to-c bot requested a deployment to preview February 6, 2024 15:41 Abandoned
linusnorton
linusnorton previously approved these changes Feb 7, 2024
View reviewed changes
velimd
velimd previously approved these changes Feb 7, 2024
View reviewed changes
@hmcts-jenkins-a-to-c hmcts-jenkins-a-to-c bot requested a deployment to preview February 8, 2024 09:46 Abandoned
@Andrew-McMahon7 Andrew-McMahon7 changed the title Test change to see if AKS deploy works Fix Security Vulnerabilities Feb 8, 2024
@Andrew-McMahon7 Andrew-McMahon7 dismissed stale reviews from velimd and linusnorton via 3d46ec2 August 30, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@linusnorton linusnorton linusnorton left review comments

@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

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Andrew-McMahon7 @linusnorton @zikrurR @velimd @hmcts-platform-operations