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

Create SROC two-part tariff flagging endpoint for new charge versions #1239

Merged
merged 49 commits into from
Aug 19, 2024

Conversation

Beckyrose200
Copy link
Contributor

@Beckyrose200 Beckyrose200 commented Aug 7, 2024

https://eaflood.atlassian.net/browse/WATER-4622
https://eaflood.atlassian.net/browse/WATER-4588

This is a part of the work for two-part tariff supplementary billing. A licence can be added to a two-part tariff supplementary bill run in 5 ways. Editing a return, adding a historic charge version, removing a licence from the annual two-part tariff bill run, recalculating a bill and if a licence is lapsed or revoked.
Out of these 5 ways, 4 of them are in the legacy code base. To keep the new code for the supplementary billing within our system code base, we have decided to create a new endpoint that the legacy code can hit to calculate if the changed licence needs to be added to a supplementary bill run. This PR sets up the endpoint and flags a licence in need of a supplementary bill run when a historic charge version is added.

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

This is apart of the work for two-part tariff supplementary billing. A licence can be added to a two-part tariff supplementary bill run in 5 ways. Editing a return, adding a historic charge version, removing a licence from the annual two-part tariff bill run, recalculate a bill and if a licence is lapsed or revoked. Out of these 5 ways, 4 of them are in the legacy code base. To keep the new code for the supplementary billing within our system code base, we have decided to create a new endpoint that the legacy code can hit to calculate if the changed licence needs to be added to a supplementary bill run. This PR is setting up the endpoint.
@Beckyrose200 Beckyrose200 added the enhancement New feature or request label Aug 7, 2024
@Beckyrose200 Beckyrose200 self-assigned this Aug 7, 2024
@Beckyrose200 Beckyrose200 changed the title Create SROC two-part tariff flagging endpoint Create SROC two-part tariff flagging endpoint for new charge versions Aug 13, 2024
@Beckyrose200 Beckyrose200 marked this pull request as ready for review August 13, 2024 09:29
Copy link
Member

@Cruikshanks Cruikshanks left a comment

Choose a reason for hiding this comment

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

Let me know if there are any issues, questions or challenges to these suggestions. All are welcome!

app/controllers/licences.controller.js Outdated Show resolved Hide resolved
app/controllers/licences.controller.js Outdated Show resolved Hide resolved
@Beckyrose200 Beckyrose200 marked this pull request as draft August 14, 2024 13:00
Beckyrose200 added a commit to DEFRA/water-abstraction-service that referenced this pull request Aug 15, 2024
DEFRA/water-abstraction-system#1239

During recent refactoring for the Supplementary billing flag process, the route was updated to better reflect the naming conventions in the repo. This PR is to update the routes when the service is called.
@Beckyrose200 Beckyrose200 marked this pull request as ready for review August 16, 2024 14:38
Beckyrose200 added a commit to DEFRA/water-abstraction-service that referenced this pull request Aug 16, 2024
DEFRA/water-abstraction-system#1239

During recent refactoring for the Supplementary billing flag process, the route was updated to better reflect the naming conventions in the repo. This PR is to update the routes when the service is called.
Copy link
Member

@Cruikshanks Cruikshanks left a comment

Choose a reason for hiding this comment

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

Hopefully just some minor tweaks plus the change we discussed.

Copy link
Member

@Cruikshanks Cruikshanks left a comment

Choose a reason for hiding this comment

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

@Beckyrose200 Beckyrose200 merged commit 5a8d0b7 into main Aug 19, 2024
6 checks passed
@Beckyrose200 Beckyrose200 deleted the supp-flag-end-point branch August 19, 2024 12:30
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