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.

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

feat: ENG-283 Nav update #638

Draft
wants to merge 101 commits into
base: master
Choose a base branch
from
Draft

feat: ENG-283 Nav update #638

wants to merge 101 commits into from

Conversation

AndyEPhipps
Copy link
Contributor

PR Titles

To pass testing pipeline, these need to follow Conventional Commits spec:

https://www.conventionalcommits.org/en/v1.0.0/
e.g.
feat: create NewForm component
or
fix: update broken link in NewForm component

PR description

What is it doing?

Tell us what it is doing from a technical perspective.

Why is this required?

Tell us why this is required from a business/product perspective.

link to Jira ticket:

Add a link to the Jira ticket.

Quick Checklist:

  • My PR title follows the Conventional Commit spec.

  • I have filled out the PR description as per the template above.

  • I have added tests to cover new or changed behaviour.

  • I have updated any relevant documentation.

Important! - lastly, make sure to squash merge...

@AndyEPhipps AndyEPhipps changed the title Eng 2834 nav update ENG-2834: Nav update Apr 11, 2024
@AndyEPhipps AndyEPhipps changed the title ENG-2834: Nav update feat: ENG-283 Nav update Apr 11, 2024
}
},
node4Copy: 'children and young people in the UK and around the world, including with safe homes, good nutrition and access to quality healthcare and education.',
node4Amount: '4,000',
node4Image: {
file: {
url: '//images.ctfassets.net/zsfivwzfgl3t/45yv3H0XZsvsySHtutQ8vb/1fb6d1afe4037c7dccde743a747f6b4f/Test-Carousel-Icon.png'
url: '//images.ctfassets.net/zsfivwzfgl3t/6ZsS5CpukQwXRcMSUlbOM1/be31c7ff09891d14232e7c3dbe9fa8a2/028-love_1.png'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just fixing these dead image links while I'm here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant