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

Match CV data to returns data for 2PT billing #328

Merged
merged 8 commits into from
Aug 2, 2023
Merged

Conversation

Jozzey
Copy link
Contributor

@Jozzey Jozzey commented Aug 1, 2023

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

First iteration of mathcing returns to charge versions. This covers the first example which is a one-to-one match.

We also have updated the results to include charge versions, charge elements and charge purposes. We intend to refine this in future iterations but we are adding them to confirm the matching algorithm is working correctly.

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

First iteration of mathcing returns to charge versions. This covers the first example which is a one-to-one match.

We also have updated the results to include charge versions, charge elements and charge purposes. We intend to refine this in future iterations but we are adding them to confirm the matching algorithm is working correctly.
@Jozzey Jozzey added the enhancement New feature or request label Aug 1, 2023
@Jozzey Jozzey marked this pull request as ready for review August 1, 2023 16:21
@Jozzey Jozzey requested review from Cruikshanks and StuAA78 August 1, 2023 16:21
@Cruikshanks Cruikshanks marked this pull request as draft August 2, 2023 06:34
@Cruikshanks
Copy link
Member

We just need to add unit tests for the new model hence put back to draft.

@Jozzey Jozzey requested review from Cruikshanks and StuAA78 August 2, 2023 14:52
@Jozzey Jozzey marked this pull request as ready for review August 2, 2023 14:52
Cruikshanks
Cruikshanks previously approved these changes Aug 2, 2023
@Jozzey Jozzey merged commit 1241d85 into main Aug 2, 2023
@Jozzey Jozzey deleted the simple-return-matching branch August 2, 2023 15:06
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.

4 participants