Skip to content

V1.0.0.1 release#249

Merged
jirhiker merged 73 commits intoproductionfrom
staging
Apr 28, 2026
Merged

V1.0.0.1 release#249
jirhiker merged 73 commits intoproductionfrom
staging

Conversation

@jirhiker
Copy link
Copy Markdown
Member

@jirhiker jirhiker commented Apr 21, 2026

V1.0.0.1 release of the UI

TylerAdamMartinez and others added 27 commits April 10, 2026 10:56
[BDMS-690 & BDMS-689]: Update the columns on the Wells list page to include contact info
Remove Field Event History component
- Move location info from CoreWellInfo to InteractiveSatelliteMap (renamed to Location)
- Move elevation info from CoreWellInfo and WellSummaryBar to Physical Properties
- Convert CoreWellInfo to compact 3-column stat bar (Hole Depth, Well Depth, Measuring Point)
- Standardize ConstructionInfo, PhysicalProperties, and GeologyInformation to inline text style
- Add site_name to IWell and surface it as Contacts card heading
- Update IWellDetails to replace removed fields with field_events list
- Expand IFieldEvent with nested field_event_participants and contact types
- Derive first visit participants from oldest field event (last in desc-sorted list)
- Wire firstVisitParticipants into MonitoringInfoCard as First Visit Staff
- Resolve merge conflict in well-show.tsx
…eld_events data

Add nested sample/observation types to IFieldActivity and field_activities to
IFieldEvent. Derive recentObservations (flattened from all field_events) and
latestSample (from newest field event) in well-show, replacing the empty
placeholder props previously passed to both components.
The paged field_events list (25 newest) does not include the oldest visit for
wells with many field events. Consume the new first_field_event field returned
by the API to reliably source first-visit participants regardless of history
length.

Also fixes participant contact name rendering: the API returns name/organization
on ContactResponse, not contact_name/contact_organization.
Reorganize card into two sections: status fields above the divider, visit
history below. Add Last Visit Date from newest field event. Relabel duration
as Measured For. Add bullet points to First Visit Staff list.
Copy link
Copy Markdown

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

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.

Reviewed commit: 8bd53424c5

ℹ️ 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 src/pages/ocotillo/thing/well-show.tsx Outdated
Comment on lines +109 to +110
.flatMap((sample) => sample.observations ?? [])
.filter((obs) => obs.observation_datetime != null)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Restrict recent observations to water-level measurements

recentObservations is built from every sample observation that has a timestamp, so non-water-level measurements (for example chemistry observations) are now mixed into data later used by both RecentWaterLevelObservationsCard and WellPDFDownloadButton. For wells with mixed observation types, this causes the “Recent Water Level Observations” table and PDF summary fields to show irrelevant rows or blank depth values. Filter to groundwater-level observations (or at least rows with depth_to_water_bgs) before sorting.

Useful? React with 👍 / 👎.

Comment on lines +119 to +122
const newestEvent = fieldEvents[0]
if (!newestEvent) return undefined
const firstActivity = newestEvent.field_activities?.[0]
return firstActivity?.samples?.[0] ?? undefined
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Compute latest sample across all event activities

latestSample currently assumes the newest sample is always fieldEvents[0] -> field_activities[0] -> samples[0], which is only correct if all three arrays are guaranteed ordered and non-empty. If the newest event’s first activity has no samples (or activities are ordered differently), the PDF loses/uses the wrong WL method even though valid recent samples exist elsewhere in the response. Select the latest sample by scanning all event activities and comparing sample/event dates.

Useful? React with 👍 / 👎.

@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-staging-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-staging-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

1 similar comment
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-staging-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

Drive thing_id from React Query queryKey inside the hydrograph queryFn, forward AbortSignal through ocotillo getList and axios, show loading while the query is pending, and remount the chart when the well id changes. Add .cursor to gitignore for local Cursor metadata.
WellList passes name_contains (not query) so the staging API can filter by
substring on Thing.name. ListPage accepts optional searchAriaLabel for screens
that customize server-side search.

Well list placeholder and aria text describe searching by well name instead of
search all records.
@jirhiker jirhiker changed the title V1 release V1.0.0.1 release Apr 27, 2026
BDMS-763 Improve well attachments grid and file table.
BDMS-773 Fix hydrograph stale data when switching wells
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-staging-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

2 similar comments
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-staging-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-staging-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

jirhiker and others added 2 commits April 27, 2026 14:33
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-staging-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

…gon-tool-on-map-doesn-t-update-the-list-on-right-side

feat(map): filter features by selection polygons and update dependencies in effects
@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-staging-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@jirhiker jirhiker merged commit 98e8d06 into production Apr 28, 2026
4 of 5 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.

3 participants