Skip to content

CDA-75 Adds flow unit field to water user contract account object and…#98

Merged
adamkorynta merged 2 commits intoHydrologicEngineeringCenter:mainfrom
rma-bryson:bugfix/CDA-75-add-pump-account-units
Feb 2, 2026
Merged

CDA-75 Adds flow unit field to water user contract account object and…#98
adamkorynta merged 2 commits intoHydrologicEngineeringCenter:mainfrom
rma-bryson:bugfix/CDA-75-add-pump-account-units

Conversation

@rma-bryson
Copy link
Contributor

@rma-bryson rma-bryson commented Jan 28, 2026

… updated unit conversion on store to use that if it exists.

db fix for CDA issue: USACE/cwms-data-api#1180
fixes #96

@rma-bryson rma-bryson marked this pull request as draft January 28, 2026 23:40
@rma-bryson rma-bryson force-pushed the bugfix/CDA-75-add-pump-account-units branch 2 times, most recently from 88cad57 to 80196df Compare January 29, 2026 23:06
@rma-bryson rma-bryson marked this pull request as ready for review January 30, 2026 20:42
CDA-75 adds unit test for water supply to test_grants

CDA-75 adds unit test to build.xml

CDA-75 adds unit test for pump flow unit conversion

CDA-75 adds new unit on retrieval of object

CDA-75 Adds flow unit field to water user contract account object and updated unit conversion on store to use that if it exists.
@rma-bryson rma-bryson force-pushed the bugfix/CDA-75-add-pump-account-units branch from 3c6f046 to 0a52f15 Compare January 30, 2026 20:43
@rma-bryson
Copy link
Contributor Author

@MikeNeilson @Enovotny @perrymanmd - tagging for review

@adamkorynta adamkorynta merged commit ececb4f into HydrologicEngineeringCenter:main Feb 2, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Water supply accounting pump flow lacks units

3 participants