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

2PT use full authorised volume rather than Nil #1404

Merged
merged 6 commits into from
Oct 17, 2024
Merged

Conversation

Jozzey
Copy link
Contributor

@Jozzey Jozzey commented Oct 11, 2024

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

From testing the business has requested that in circumstances where we can't match or determine a volume for the review stage, we use the Full Authorised Volume rather than a nil volume. Unless there are multiple returns and multiple charge versions where only some match, in which case use the volumes available.

This PR will resolve this issue.

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

From testing the business has requested that in circumstances where we can't match or determine a volume for the review stage, that we use the Full Authorised Volume rather than a nil volume. Unless there are multiple returns and multiple charge versions where only some match, in which case uses volumes available.

This PR will resolve this issue.
@Jozzey Jozzey added the enhancement New feature or request label Oct 11, 2024
@Jozzey Jozzey self-assigned this Oct 11, 2024
Jozzey and others added 3 commits October 15, 2024 12:49
When a charge reference has not been matched to a return this code will allocate the elements authorised quantity to its allocated quantity. The quantity assigned to the elements cannot exceeded the references authorised quantity.
@Jozzey Jozzey marked this pull request as ready for review October 15, 2024 14:58
@Jozzey Jozzey merged commit d4939f1 into main Oct 17, 2024
6 checks passed
@Jozzey Jozzey deleted the use-authorised-not-nil branch October 17, 2024 10:35
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.

2 participants