Skip to content

Conversation

@kr8n3r
Copy link
Contributor

@kr8n3r kr8n3r commented Jan 27, 2026

What

Update the guidance on the ‘Add a new service’ screen.
Explicitly tell users that the service name is for us (Notify), and not for recipients.

Update the guidance on the ‘Confirm your service is unique’ page.
Explicitly tell users that the service name is for us (Notify), and not for recipients.
Replace the old instructions.

Trello card
Trello card

Changes include:

  • content updates
  • removal of conditionally displayed content examples
  • stylistic tweaks
  • updates to tests

Why

In order to encourage users to choose a good service name (one which doesn’t cause problems when they request to go live), we need to change the name-your-service guidance.
We need to update the content on the ‘Confirm your service is unique’ page so that it does not contradict the ‘Name your service’ page.

Visuals

Before

notify localhost_6012_add-service (4) notify localhost_6012_add-service (3) Screenshot 2026-01-30 at 08 54 05 Screenshot 2026-01-30 at 11 25 16

After

notify localhost_6012_add-service (2) notify localhost_6012_add-service (3) Screenshot 2026-01-29 at 19 21 51 Screenshot 2026-01-30 at 11 25 05

@kr8n3r kr8n3r force-pushed the update-name-your-service-guidance branch from 7e536c1 to f04f9e0 Compare January 27, 2026 18:43
@kr8n3r kr8n3r changed the title WIP: Update "name your service" page Update "Add a service" page Jan 27, 2026
@kr8n3r kr8n3r marked this pull request as ready for review January 27, 2026 18:44
@kr8n3r kr8n3r force-pushed the update-name-your-service-guidance branch from f04f9e0 to d94db40 Compare January 29, 2026 09:56
@kr8n3r kr8n3r changed the title Update "Add a service" page Update "Add a service" and "confirm your service is unique" pages Jan 30, 2026
@kr8n3r kr8n3r force-pushed the update-name-your-service-guidance branch from e426ea1 to 52faa25 Compare January 30, 2026 09:15
@tombye tombye self-assigned this Jan 30, 2026
@kr8n3r kr8n3r force-pushed the update-name-your-service-guidance branch 2 times, most recently from 9eec9f6 to 2efb399 Compare January 30, 2026 10:34
@kr8n3r kr8n3r force-pushed the update-name-your-service-guidance branch from 685f0f9 to 47c7386 Compare January 30, 2026 10:52
@kr8n3r kr8n3r changed the title Update "Add a service" and "confirm your service is unique" pages Update "Add a service", "confirm your service is unique" and "change service name" pages Jan 30, 2026
@kr8n3r kr8n3r force-pushed the update-name-your-service-guidance branch 7 times, most recently from 6aa026e to e14767c Compare February 3, 2026 10:49
This updates content based on the content document
(see Trello ticket) and makes the following markup
changes:

- removes conditional content from bullet points
- adds a visible legend styled as h2 to the
`organisation_type` to bring the section into
prominence
- adds organisation_type form paramaters from
python into the html code
- adds a 'details' element with new content
'CreateNhsServiceForm' classes

- Update the service name field label
- Remove the form label from OrganisationTypeField
class as we now control the legend text in the
view.
Update tests that check for H1 text
Remove the test that was testing conditional
content in the bullet point examples, as
they're no loneger present.
Update according to content doc -
https://docs.google.com/document/d/1ZItE5zQzkkdraRHrRml6AXlIldXPfaO0ODS2VUYLMvI/edit?tab=t.0
Remove conditional content
Update test and remove redundant ones
@kr8n3r kr8n3r force-pushed the update-name-your-service-guidance branch from e14767c to 48601c1 Compare February 3, 2026 16:30
Copy link
Contributor

@tombye tombye left a comment

Choose a reason for hiding this comment

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

The code looks good and the content matches that in the docs. Also thanks for making a change I asked for outside of this review (removing a h2 above some radios and using their legend instead) 👍🏻

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.

4 participants