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

Refactor existing supplementary billing flags service #1483

Merged
merged 26 commits into from
Nov 20, 2024

Conversation

Beckyrose200
Copy link
Contributor

@Beckyrose200 Beckyrose200 commented Nov 14, 2024

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

As part of our efforts to clean up the legacy code base and ensure that our existing functionality is working, we have been refactoring and migrating the logic for supplementary billing flags.
This PR focuses on refactoring the existing legacy code to handle flagging for all three types (pre-sroc, sroc and two-part tariff).

https://eaflood.atlassian.net/jira/software/c/projects/WATER/boards/96

As part of our efforts to clean up the legacy code base and ensure that our existing functionality is working, we have been refactoring and migrating the logic for supplementary billing flags. This PR is focused on migrating the mechanism for flagging a licence when it is removed from a bill run in a ready status.
@Beckyrose200 Beckyrose200 added the housekeeping Refactoring, tidying up or other work which supports the project label Nov 14, 2024
@Beckyrose200 Beckyrose200 self-assigned this Nov 14, 2024
@Beckyrose200 Beckyrose200 force-pushed the refactor-remove-bill-licence-flags branch from 3abbcb6 to 89c08a7 Compare November 14, 2024 16:30
@Beckyrose200 Beckyrose200 changed the title Refactor remove bill licence supplementary billing flags Refactor existing supplementary billing flags service Nov 14, 2024
@Beckyrose200 Beckyrose200 marked this pull request as ready for review November 15, 2024 14:20
@Beckyrose200 Beckyrose200 requested a review from StuAA78 November 18, 2024 21:59
Copy link
Contributor

@StuAA78 StuAA78 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 4feb786 into main Nov 20, 2024
6 checks passed
@Beckyrose200 Beckyrose200 deleted the refactor-remove-bill-licence-flags branch November 20, 2024 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Refactoring, tidying up or other work which supports the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants