Skip to content

NO TICKET: Transfer#147

Merged
jirhiker merged 52 commits into
stagingfrom
transfer
Sep 26, 2025
Merged

NO TICKET: Transfer#147
jirhiker merged 52 commits into
stagingfrom
transfer

Conversation

@jirhiker
Copy link
Copy Markdown
Member

Why

This PR addresses the following problem / context:

  • The transfer required modification to the schema. nma_pk_location was set as unique but there are non-unique LocationIDs in NMAquifer

How

Implementation summary - the following was changed / added / removed:

  • removed unique flag from nma_pk_location

Notes

Any special considerations, workarounds, or follow-up work to note?

  • includes updated transfer_asset function

jirhiker and others added 30 commits September 10, 2025 23:18
# Conflicts:
#	core/app.py
#	pyproject.toml
#	transfers/asset_transfer.py
#	transfers/contact_transfer.py
#	transfers/thing_transfer.py
#	transfers/transfer.py
#	transfers/well_transfer.py
#	uv.lock
# Conflicts:
#	pyproject.toml
#	transfers/contact_transfer.py
#	transfers/transfer.py
#	transfers/well_transfer.py
#	uv.lock
# Conflicts:
#	transfers/transfer.py
#	transfers/waterlevels_transfer.py
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Files with missing lines Coverage Δ
core/initializers.py 72.00% <100.00%> (ø)
db/location.py 94.87% <100.00%> (ø)
services/lexicon_helper.py 100.00% <ø> (ø)

Comment thread transfers/waterlevels_transfer.py
@jirhiker jirhiker merged commit 4108f74 into staging Sep 26, 2025
2 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.

4 participants