Skip to content

feat: add water level fields and scenario to well inventory feature#286

Merged
jirhiker merged 1 commit into
well-inventory-csvfrom
cm-well-inventory-feature-update
Dec 10, 2025
Merged

feat: add water level fields and scenario to well inventory feature#286
jirhiker merged 1 commit into
well-inventory-csvfrom
cm-well-inventory-feature-update

Conversation

@chasetmartin

Copy link
Copy Markdown
Collaborator

Why

This PR addresses the following problem / context:

  • Add water level entry fields to as optional fields during a well inventory

How

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

Notes

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

  • Adds one validation scenario for when any water level field is filled out - requiring all other water level fields.
  • level_status lexicon was highlighted as important to users to reflect the USGS use
  • data_quality lexicon was also reflected as important to users to to reflect the USGS use

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tests/features/well-inventory-csv.feature
Comment thread tests/features/well-inventory-csv.feature
@jirhiker jirhiker merged commit 0a617ff into well-inventory-csv Dec 10, 2025
4 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the cm-well-inventory-feature-update branch February 26, 2026 18:28
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.

2 participants