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 Maternity, Paternity and Adoption calculators #7090

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

baisa
Copy link
Contributor

@baisa baisa commented Feb 27, 2025

Trello card

As part of 2025 uprating. All rates increase from £184.03 to £187.18 and LEL is changing from £123 to £125 from 6 April 2025. This commit also includes fix, changing incorrect date uprating_date(2023) to 2024 for the previous year.

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Follow these steps if you are doing a Rails upgrade.

@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7090 February 27, 2025 13:52 Inactive
@baisa baisa force-pushed the maternity-paternity-calculator branch from 7e4d394 to 34583c9 Compare February 27, 2025 15:51
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7090 February 27, 2025 15:51 Inactive
@baisa baisa marked this pull request as ready for review February 27, 2025 15:57
As part of 2025 uprating. All rates increase from £184.03 to £187.18 and LEL is changing from £123 to £125 from 6 April 2025. This commit also includes fix, changing incorrect date uprating_date(2023) to 2024 for the previous year.
Copy link
Contributor

@syed-ali-tw syed-ali-tw left a comment

Choose a reason for hiding this comment

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

Code changes looks good to me, have not manually tested the flow for the calculator, guessing you would have done it already

@baisa baisa added the DO NOT MERGE Needs further approval before release label Mar 4, 2025
@baisa
Copy link
Contributor Author

baisa commented Mar 4, 2025

Code changes looks good to me, have not manually tested the flow for the calculator, guessing you would have done it already

Thank you. I have tested it but it also awaits fact check, so I have added 'Do not merge' label until this is approved by department.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Needs further approval before release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants