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

Silo rebilling from supplementary #854

Merged
merged 5 commits into from
Mar 24, 2024
Merged

Silo rebilling from supplementary #854

merged 5 commits into from
Mar 24, 2024

Conversation

Cruikshanks
Copy link
Member

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

When we initially delivered SROC supplementary billing we were required to continue to support handling reissuing bills.

But the feature was put behind a feature flag so we could ship supplementary and then worry about reissuing.

At the time we pointed out that in the life of WRLS, only 8 bills had ever been re-issued. Yet it was one of the most complex processes to support.

It turns out the billing & data team had also come to the same conclusion. There are things they have to be aware of and features in the UI they need to enable reissuing to happen. Alternatively, they can just call SSCL and request they reissue the bill from SOP.

Because of this, they've held off asking us to enable the feature.

So, it has been more than 6 months and reissuing still hasn't been enabled in production. In that time we've made many other changes where we've disregarded reissuing to avoid needless complexity and our belief a decision will be made never to enable it.

Should that assumption be wrong we'd need to make a suite of changes to support it, for example, in how we display things in the billing views. We are making a series of updates to the supplementary billing engine to implement the improvements we implemented in SROC annual billing. That refactoring is a prime opportunity to silo the reissuing from the engine and the complexity it adds.

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

When we initially delivered SROC supplementary billing we were required to continue to support handling reissuing bills.

But the feature was put behind a feature flag so we could ship supplementary and then worry about reissuing.

At the time we pointed out that in the life of WRLS only 8 bills had ever been re-issued. Yet it was one of the most complex processes to support.

It turns out the billing & data team had also come to the same conclusion. There were things they had to be aware of and features in the UI they need to enable for reissuing to happen. Alternatively, they can just call SSCL and request they reissue the bill from SOP.

Because of they've held off asking us to enable the feature.

So, it has been more than 6 months and reissuing still hasn't been enabled in production. In that time we've made many other changes where we've disregarded reissuing to avoid needless complexity and our belief a decision will be made never to enable it.

Should that assumption be wrong we'd need to make a suite of changes to support it, for example, in how we display things in the billing views. We are making a series of updates to the supplementary billing engine to implement the improvements we implemented in SROC annual billing. That refactoring is a prime opportunity to silo the reissuing from the engine and the complexity it adds.
@Cruikshanks Cruikshanks added the housekeeping Refactoring, tidying up or other work which supports the project label Mar 24, 2024
@Cruikshanks Cruikshanks self-assigned this Mar 24, 2024
We move the services that currently support reissuing a bill (rebilling) into their own folder and remove the call to them in the supplementary bill run process.
@Cruikshanks Cruikshanks marked this pull request as ready for review March 24, 2024 16:00
@Cruikshanks Cruikshanks merged commit fcbe07f into main Mar 24, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the side-line-rebilling branch March 24, 2024 16:00
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.

1 participant