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

Update default index page #423

Merged
merged 5 commits into from
Nov 22, 2024
Merged

Update default index page #423

merged 5 commits into from
Nov 22, 2024

Conversation

frankieroberto
Copy link
Contributor

@frankieroberto frankieroberto commented Nov 19, 2024

Following #419, this updates the default index page to give clearer guidance on how to:

  • set the service name (which is now set to "Service name goes here" by default)
  • edit the index page
  • use the external links and included example templates

Inspired by the GOV.UK Prototype kit default index page.

Screenshots

Before After
index-page-before index-page-after

Checklist

  • CHANGELOG entry

This aims to give clearer guidance on how to:

* set the service name (which is now set to "Service name goes here" by default)
* edit the index page
* use the external links and included example templates
@vickytnz
Copy link
Contributor

I think Sara may need to review this, on the same page we have:

  • NHS.UK prototype kit
  • NHS Prototype kit

I think that both now should be NHS prototype kit (without the .UK and lower case) but don't really know right now.

vickytnz
vickytnz previously approved these changes Nov 21, 2024
@frankieroberto
Copy link
Contributor Author

@vickytnz thanks, have updated from main now with your changes to capitalisation and NHS.UK -> NHS. Could you re-review?

Copy link
Contributor

@vickytnz vickytnz left a comment

Choose a reason for hiding this comment

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

All looks good - I've also fixed the broken link in the footer that we all missed when removing the example page 🙈

@frankieroberto frankieroberto merged commit ba5f75d into main Nov 22, 2024
3 checks passed
@frankieroberto frankieroberto deleted the update-default-index-page branch November 22, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants