V1.0.1#279
Conversation
BDMS-777: Update the PDF preview and download button so that they only require view access.
Parse Ocotillo and AMP well show URLs so $pageview includes well_id, page_template, and well_detail_area for breakdowns. Extend feature_used on Ocotillo WellShow with well_detail_area.
…n replay. Emit global_search after API and docs searches with query, result_count, has_results, and had_error. Mask other inputs in replay but show the palette field via data-posthog-unmask-search and maskInputFn.
BDMS-562: PostHog well detail routing and command palette search analytics
Contacts that represent companies rather than individuals have no name value, causing blank labels across the UI and in search results. Add a getContactDisplayName utility that falls back to organization when name is absent, and apply it to the wells list, contacts list, contact show page, and the well detail contacts card. Document the logic and deferred follow-up work in feature_files/contact-display-name.md.
BDMS-775: Remove DataGrid Export and Type Contact Link Click Handler
…ntact-Name Show organization name for contacts without a personal name
BDMS-776: Increased the export hook pageSize from 20 to 500 on well list page
BDMS-788 Patch broken link in home page
Preview DeploymentPreview URL: https://preview-staging-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
|
@jirhiker, is this release only for bug fixes? I ask because I'm wondering whether this release depends on |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6e9d2970d7
ℹ️ 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".
| import { FieldCompilationNotesPdf } from '@/components/pdf/FieldCompilationNotesPdf' | ||
| import type { IContact, IWell } from '@/interfaces/ocotillo' | ||
| import { formatContactPhones } from '@/components/pdf/fieldCompilationPhoneFormatter' | ||
| import * as pdfjsLib from 'pdfjs-dist/legacy/build/pdf.mjs' |
There was a problem hiding this comment.
The new pdfjs-dist import runs in Vitest’s Node environment, but this commit pins pdfjs-dist@^5.7.284, whose package metadata requires Node >=22.13.0; our Vitest workflow still runs on Node 20.x (.github/workflows/CI_vitest.yml, line 17). In that CI context, loading this module causes the test suite to fail before assertions run, so this change breaks the default test pipeline unless CI is upgraded to Node 22+ or the parser/version is downgraded.
Useful? React with 👍 / 👎.
|
@TylerAdamMartinez it is intended for only prioritized bug fixes. I changed this pr to draft as we currently don't have a time set for a new production hot fix release |
3 participants
Why
This PR addresses the following problem / context:
How
Implementation summary - the following was changed / added / removed:
Notes
Any special considerations, workarounds, or follow-up work to note?