Skip to content

Add neonatalcare leave earnings types to EarningsRate type #709

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

Conversation

asdasdasdasdasd-rgb
Copy link

Description

Add a new value of earnings types to the earnings rates component. This affects the request and response bodies for the POST and GET endpoints for EarningsRates, and the GET response for endpoint EarningsRates/{id}.

Release Notes

Screenshots (if appropriate):

Types of Changes

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Copy link

github-actions bot commented Apr 6, 2025

PETOSS-718

Copy link

github-actions bot commented Apr 6, 2025

Thanks for raising an issue, a ticket has been created to track your request

sangeet-joy-tw

This comment was marked as duplicate.

@sangeet-joy-tw sangeet-joy-tw self-requested a review April 7, 2025 06:54
@sangeet-joy-tw sangeet-joy-tw dismissed their stale review April 7, 2025 06:56

Will put it as a comment

@@ -6901,6 +6901,8 @@ components:
- StatutoryBereavementPay
- StatutoryMaternityPay
- StatutoryMaternityPayNonPensionable
- StatutoryNeonatalCarePay
- StatutoryNeonatalCarePayNonPensionable
- StatutoryPaternityPay
Copy link
Contributor

Choose a reason for hiding this comment

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

@asdasdasdasdasd-rgb

We have reviewed the PR and generated the changes locally for all SDKs however for Dotnet SDK it is a breaking change due to enum numberings are changed as depicted in the screenshot.

Would it be possible to add these two properties - StatutoryNeonatalCarePay, StatutoryNeonatalCarePayNonPensionable at the end of earningsType property in the PR.

image

sangeet-joy-tw

This comment was marked as duplicate.

@sangeet-joy-tw
Copy link
Contributor

Duplicate PR - #710

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.

2 participants