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

Enter abstraction period dates for return set up #610

Conversation

Demwunz
Copy link
Contributor

@Demwunz Demwunz commented Dec 19, 2023

https://eaflood.atlassian.net/browse/WATER-4274

This is a basic page with a form to enter abstraction period dates for a return set up.

This is a stub page.

https://eaflood.atlassian.net/browse/WATER-4274

This is a basic page with a form to enter abstraction period dates for a return set up.

This is a stub page.
@Demwunz Demwunz force-pushed the WATER-4274-enter-abstraction-period-dates-for-return-set-up-basic-page branch from 90af4ac to d29ab04 Compare December 20, 2023 14:57
@Demwunz Demwunz marked this pull request as ready for review December 28, 2023 09:35
{% from "govuk/components/back-link/macro.njk" import govukBackLink %}
{% from "govuk/components/button/macro.njk" import govukButton %}

{% set title = "Enter the abstraction period for the return requirement
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't mind doing it this way but we need to be consistent. some of the pages have the text in the H1 element. so we either need to change the others to this way or make this the same as those.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@robertparkinson I've created another branch and moved all title strings in other files to use the set.
I'm lifting this from the prototype, so it's less friction when copying across nunjucks code.

@Demwunz Demwunz merged commit 3a7a60b into main Dec 29, 2023
6 checks passed
@Demwunz Demwunz deleted the WATER-4274-enter-abstraction-period-dates-for-return-set-up-basic-page branch December 29, 2023 15:59
@Cruikshanks Cruikshanks added the enhancement New feature or request label Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants