Skip to content

Wells list: prioritize key columns and add column descriptions#264

Merged
jeremyzilar merged 6 commits intostagingfrom
feature/wells-list-columns-reorder
Apr 28, 2026
Merged

Wells list: prioritize key columns and add column descriptions#264
jeremyzilar merged 6 commits intostagingfrom
feature/wells-list-columns-reorder

Conversation

@jeremyzilar
Copy link
Copy Markdown
Contributor

@jeremyzilar jeremyzilar commented Apr 27, 2026

Waiting on DataIntegrationGroup/OcotilloAPI#665 to be merged

What is changing

Reorders the wells DataGrid so the first visible columns are Name, Site name, Monitoring, and Created At, then the rest of the fields in a familiar order. Adds a site_name column (from IWell.site_name).

Column descriptions

Each GridColDef now has a description string for clarity in the MUI X column header metadata and related UI (for example the columns management experience where supported).

Reorder DataGrid columns so Name, Site name, Monitoring, and Created At appear
first. Add site_name column with empty-string fallback. Set description on each
column for MUI header accessibility and columns panel.
Use formatAppDate instead of formatAppDateTime for the Created At column and
tighten column width.
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-wells-list-columns-reorder-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-wells-list-columns-reorder-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

Use API site_name with NMBGMR alternate_id fallback to match backend
Thing.site_name. Apply on wells list column and well show ContactsCard.
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-wells-list-columns-reorder-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

@jeremyzilar Updating the sort breaks in preview.

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

Captura de pantalla 2026-04-27 a la(s) 16 46 25

@jeremyzilar
Copy link
Copy Markdown
Contributor Author

Thank you @TylerAdamMartinez -- will fix

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

Instead of trying to patch sorting. Why not remove it for now....

@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-wells-list-columns-reorder-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

…lter limits

Enable sorting on Associated Sites, Aquifers, and Contacts where the API defines rules and add column descriptions for sort semantics. Update data-tables.md for MUI X Community single column filter in the panel, multi-filter capability on the API side, and refresh sortability in the column tables. Clarify site name description and simplify README deployment hosting examples.
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-wells-list-columns-reorder-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

@jeremyzilar I'm taking it that DataIntegrationGroup/OcotilloAPI#665 needs to be merged first.

After each successful thing/water-well details fetch, log a JSON snapshot and pretty-printed copy so the full response is inspectable in DevTools in dev, staging, and production.
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-wells-list-columns-reorder-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@jeremyzilar jeremyzilar merged commit ffb53d7 into staging Apr 28, 2026
3 of 5 checks passed
@jeremyzilar jeremyzilar deleted the feature/wells-list-columns-reorder branch April 28, 2026 15:59
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