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 pat leave calculations for NI #6766

Merged
merged 8 commits into from
May 21, 2024

Conversation

mtaylorgds
Copy link
Contributor

@mtaylorgds mtaylorgds commented May 1, 2024

Update the paternity leave calculator to handle employees living in Northern Ireland.

The rules for employees living in Scotland, England, and Wales changed at the start of the new tax year in 2024. However, these changes do NOT apply to employees living in Northern Ireland. Update the calculator to handle Northern Ireland employees by inserting a new question into the flow (in two places—one for the paternity leave flow, and one for paternity adoption leave flow).

There are also a few minor text changes.

Trello ticket

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

Follow these steps if you are doing a Rails upgrade.

Attempt to make the tests slightly easier to read in preparation for
 adding in context blocks for where the employee lives.
Update the paternity leave calculator for employers to handle employees
 who live in Northern Ireland—the rules for Northern Ireland-based
 employees remains the same, they did not change on the 6th April 2024.
Attempt to make the tests slightly easier to read in preparation for
 adding in context blocks for where the employee lives.
Update the paternity adoption leave calculator for employers to handle
 employees who live in Northern Ireland—the rules for Northern
 Ireland-based employees remains the same, they did not change on the
 6th April 2024.
Insert a question into the paternity and paternity adoption calculator
 flows asking where the employee lives. For paternity leave this comes
 after the first question (where they indicate it is for paternity
 leave), and for paternity adoption leave it comes after second question
 (i.e. as soon as we know it's for paternity adoption).

Update the comments labelling the questions with numbers (including
 re-indexing from one, instead of zero, to match the maternity
 calculator flow).
The calculator now handles employees applying for paternity and
 paternity adoption leave who live in Northern Ireland.

The use of "eg" has been replaced with "for example" to fit GOV.UK style.
Simplify the outcome text when an employee is not entitled to statutory
 paternity pay.
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-6766 May 1, 2024 14:37 Inactive
@mtaylorgds mtaylorgds changed the title Update pat leave calculations for NI [DO NOT MERGE UNTIL FACT-CHECKED] Update pat leave calculations for NI May 1, 2024
@mtaylorgds mtaylorgds changed the title [DO NOT MERGE UNTIL FACT-CHECKED] Update pat leave calculations for NI Update pat leave calculations for NI May 21, 2024
Copy link
Contributor

@anatron anatron left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

@mtaylorgds mtaylorgds merged commit 401e6be into main May 21, 2024
17 checks passed
@mtaylorgds mtaylorgds deleted the employer_mat_pat_calculator/update-for-ni branch May 21, 2024 09:29
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.

None yet

3 participants