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

Change where the allocated Return volume is held #445

Merged
merged 2 commits into from
Oct 3, 2023

Conversation

Jozzey
Copy link
Contributor

@Jozzey Jozzey commented Oct 3, 2023

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

In a previous PR #436 which dealt with allocating the Return volumes to Charge Elements. It was assumed that the billableAnnualQuantity data item could be used to hold the Return volumes allocated to a Charge Element. However, in very few cases, this could already contain data which will cause the Return volume allocation process to not work properly.

Therefore this PR will create a new data item on the Charge Element called allocatedReturnVolume that will be used to hold any Return volumes allocated to a particular Charge Element.

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

In a previous PR #436 which dealt with allocating the Return volumes to Charge Elelemts. It was assumed that the `billableAnnualQuantity` data item could be used to hold the Return volumes allocated to a Charge Element. However, in a very few cases this could already contain data which will cause the Return volume allocation process to not work properly.

Therefore this PR will create a new data item on the Charge Element called `allocatedReturnVolume` that will be used to hold any Return volumes allocated to a particular Charge Element.
@Jozzey Jozzey added the bug Something isn't working label Oct 3, 2023
@Jozzey Jozzey self-assigned this Oct 3, 2023
@Jozzey Jozzey marked this pull request as ready for review October 3, 2023 11:31
@Jozzey Jozzey requested a review from Cruikshanks October 3, 2023 11:31
@Jozzey Jozzey merged commit ab63e20 into main Oct 3, 2023
6 checks passed
@Jozzey Jozzey deleted the allocated-return-volume-data-item-name branch October 3, 2023 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants