V1.0.0.2 #268
Conversation
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.
Use API site_name with NMBGMR alternate_id fallback to match backend Thing.site_name. Apply on wells list column and well show ContactsCard.
…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.
…p focus while typing Refine sets filterDebounceMs to 0 for server mode; override to 350ms when filterMode is server so column toolbar filters do not lose focus on each keystroke.
Add short delay to filtering
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.
…columns-reorder Wells list: prioritize key columns and add column descriptions
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ffb53d7260
ℹ️ 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".
| console.log( | ||
| `${label} (full JSON, scroll or copy this if the object above will not expand)\n${JSON.stringify(data, null, 2)}` | ||
| ) |
There was a problem hiding this comment.
Gate well details debug logs behind a development check
The queryFn now logs the entire thing/water-well/{id}/details payload on every successful fetch, including a fully stringified JSON dump, in all environments. In production this exposes contact and well-detail data to any user with browser console access (and to any console-capture tooling), and it adds avoidable serialization overhead on each page load. Please restrict this logging to explicit debug/dev mode or remove it before release.
Useful? React with 👍 / 👎.
Preview DeploymentPreview URL: https://preview-staging-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
BDMS-723: Add public data acknowledgement status & refactored boolean status chips
BDMS-722: Refactor WellShow UI to Card components, consolidate physical properties, and add historic depth to water support
Preview DeploymentPreview URL: https://preview-staging-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
1 similar comment
Preview DeploymentPreview URL: https://preview-staging-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
BDMS-783 -- add contacts to CSV Export on Map pages
Preview DeploymentPreview URL: https://preview-staging-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
|
Don't merge without #271, please |
…ontrol tests. Import PKCE and JWT helpers from Auth, Http, and Jwt modules instead of the utils index so ApiFetch no longer pulls authentik-provider during module init. Partial-mock @/config with importOriginal instead of vi.importActual inside the mock factory.
BDMS-769: Convert desktop GIS help page to Markdown content & update it's content
Preview DeploymentPreview URL: https://preview-staging-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
Final release before launch