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

Determine returns matching status for licence #365

Merged
merged 9 commits into from
Aug 18, 2023
Merged

Conversation

Jozzey
Copy link
Contributor

@Jozzey Jozzey commented Aug 17, 2023

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

This PR will add a new returnsMatchingStatus data item to the Two Part Tariff algorithms output. This is calculated as follows:

  • No matching returns then mark as error
  • A return status is UNDER QUERY then mark as error (this is a boolean in the returns table)
  • A return status is NOT KEYED then mark as error (assumption made this is the received status in the returns table)
  • A return status is DUE then mark as ready
  • A return status is COMPLETE then mark as ready

There is also a possible return status of void, this will also mark as error.

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

This PR will add a new `returnsMatchingStatus` data item to the Two Part Tariff algorithms output. This is calculated as follows:
- No matching returns then mark as `error`
- A return status is `UNDER QUERY` then mark as `error`
- A return status is `NOT KEYED` then mark as `error`
- A return status is `DUE` then  mark as `ready`
- A return status is `COMPLETE` then mark as `ready`

When there are no matching returns the `returnStatuses` will be set to `Unable to match return`
@Jozzey Jozzey added the enhancement New feature or request label Aug 17, 2023
@Jozzey Jozzey self-assigned this Aug 17, 2023
@Jozzey Jozzey marked this pull request as ready for review August 18, 2023 11:30
@Jozzey Jozzey merged commit 97e0993 into main Aug 18, 2023
@Jozzey Jozzey deleted the add-matching-status branch August 18, 2023 12:10
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