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

Change date formats on check answers page in adviser funnel - DO NOT MERGE YET! #4373

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sarahcrack
Copy link
Collaborator

Trello card

https://trello.com/c/Z6odPjEh/6961-change-date-formats-on-check-answers-page-in-adviser-funnel-on-git

Context

Whilst reviewing the adviser funnel as part of the move to GIT, a few minor improvements were noticed. One of these concerns the date format used on the check answers page (/teacher-training-adviser/sign_up/review_answers) for date of birth and for the callback bookings. The date format should be e.g. 1 January 1990.

Changes proposed in this pull request

Minor tweak to AnswerHelper to format dates with :govuk_zero_pad

Guidance to review

  • check visuals meet GDS best practice on check answers page.
  • ensure this change only relates to the display on the review answers page; the format of the value passed to CRM should not be changed.

@github-actions github-actions bot added the ruby Pull requests that update Ruby code label Nov 26, 2024
@github-actions github-actions bot added the test label Nov 26, 2024
@sarahcrack sarahcrack changed the title Change date formats on check answers page in adviser funnel Change date formats on check answers page in adviser funnel - DO NOT MERGE YET! Nov 26, 2024
Copy link

@sarahcrack
Copy link
Collaborator Author

N.B. need to check whether this change to the AnswerHelper affects the passing of date values to CRM.

Copy link

sonarcloud bot commented Nov 26, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Review ruby Pull requests that update Ruby code test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant