Skip to content

[BDMS-538] Add additional required section#232

Merged
TylerAdamMartinez merged 7 commits intostagingfrom
TAM-BDMS-538
Mar 31, 2026
Merged

[BDMS-538] Add additional required section#232
TylerAdamMartinez merged 7 commits intostagingfrom
TAM-BDMS-538

Conversation

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • Some required fields on the Well Details page and Water Levels table were not included

How

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

  • Removed duplicate "Directions to the site" note from Notes component (now only shown in map section)
  • Verified required fields:
    • Casing diameter
    • Directions to the site
    • Location of the well on the site
    • Measuring point description
    • Measurement notes

Notes

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

  • "Name(s) of person who took the measurement" is currently not provided by the API (/thing/water-well/{id}/details)
  • Some missing required fields will require backend updates before they can be displayed in the UI
  • Identified missing/unimplemented required fields:
    • Names of people who inventoried the well
    • Date the site was visited
    • Date the site was inventoried
    • Name(s) of person who took the measurement (not returned from API)

@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: $PREVIEW_URL

Note: This preview uses the staging API endpoints and has auth disabled for testing.

@TylerAdamMartinez TylerAdamMartinez merged commit c1688b2 into staging Mar 31, 2026
3 of 4 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the TAM-BDMS-538 branch March 31, 2026 16:36
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: $PREVIEW_URL

Note: This preview uses the staging API endpoints and has auth disabled for testing.

1 similar comment
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: $PREVIEW_URL

Note: This preview uses the staging API endpoints and has auth disabled for testing.

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