Skip to content

BDMS-801: Update well detail page to view, preview and download uploaded digital assets#290

Merged
TylerAdamMartinez merged 9 commits into
stagingfrom
TAM-BDMS-801
Jun 12, 2026
Merged

BDMS-801: Update well detail page to view, preview and download uploaded digital assets#290
TylerAdamMartinez merged 9 commits into
stagingfrom
TAM-BDMS-801

Conversation

@TylerAdamMartinez

@TylerAdamMartinez TylerAdamMartinez commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • Users need to view all uploaded digital assets on the well detail page.
  • The attachments section previously only supported image-style previews.
  • Users need basic preview support for supported file types, including images, PDFs, and plain text files.
  • Users need a clear way to open and download attached digital assets.

How

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

  • Added preview type detection for images, PDFs, and plain text assets.
  • Replaced image-only asset filtering with previewable asset filtering based on MIME type and signed URL availability.
  • Added a reusable AssetPreview component to render image, PDF, and text previews.
  • Updated grid and slideshow views to use AssetPreview instead of directly rendering images.
  • Added Open and Download actions to the attachments table.

Notes

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

  • Browser support for previewing some formats, such as TIFF, may be limited.
  • PDFs and text files are rendered through iframes using the signed URL.
  • Assets without a signed URL are still listed in the table but cannot be previewed, opened, or downloaded from the current UI.

@TylerAdamMartinez TylerAdamMartinez self-assigned this Jun 11, 2026
@TylerAdamMartinez TylerAdamMartinez added the enhancement New feature or request label Jun 11, 2026
@github-actions

Copy link
Copy Markdown

Preview Deployment

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

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez TylerAdamMartinez requested review from chasetmartin, jeremyzilar, jirhiker and lizalino and removed request for jeremyzilar June 12, 2026 12:48
@TylerAdamMartinez TylerAdamMartinez changed the title WIP BDMS-801: Update well detail page to view, preview and download uploaded digital assets BDMS-801: Update well detail page to view, preview and download uploaded digital assets Jun 12, 2026
@TylerAdamMartinez TylerAdamMartinez marked this pull request as ready for review June 12, 2026 12:48
@github-actions

Copy link
Copy Markdown

Preview Deployment

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

Note: This preview uses the staging API endpoints.

@jeremyzilar

Copy link
Copy Markdown
Contributor
image

So this is just making it possible to show PDFs and other file types in this component as well?

@jeremyzilar

Copy link
Copy Markdown
Contributor

I see it now, on the preview:
image

Can you remove the URL column from that table? It is not a functional column, and should just be removed.

Can you make the downloaded filename consistent with the visible filename in the table? They are all downloading as AR-0025 Ocotillo Preview.JPG. Maybe it can be: AR-0025_1 Ocotillo Preview.JPG?

image

So if I want to download the image with the person standing in the snow, how would I know which image to download? How can we better tie or associate those images with the list below?

One way would be to add a download button in the bottom right corner of each image (as an icon that only appears on hover). That way, I can just download from the image itself. Another way would be to add the filename as caption text below each image. Something to noodle on.

@github-actions

Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-tam-bdms-801-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@chasetmartin chasetmartin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice, I like your implementation of the preview overlay with the download button and the filename like Jeremy suggested. Both downloads I tried functioned as expected 👍

@lizalino

Copy link
Copy Markdown

Love the overlay with download, this is looking really good!

@github-actions

Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-tam-bdms-801-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez TylerAdamMartinez merged commit 6a49156 into staging Jun 12, 2026
3 of 4 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the TAM-BDMS-801 branch June 12, 2026 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants