Skip to content

[PLAT-7588] Add file download action in dev dashboard#297

Merged
danschultz merged 1 commit intomainfrom
codex-plat-7588-file-download
Apr 8, 2026
Merged

[PLAT-7588] Add file download action in dev dashboard#297
danschultz merged 1 commit intomainfrom
codex-plat-7588-file-download

Conversation

@danschultz
Copy link
Copy Markdown
Contributor

@danschultz danschultz commented Apr 7, 2026

Summary

  • add a new dashboard API route to request file download presigned URLs
  • add a download action in each file row that opens the returned URL
  • keep file details drawer unchanged for v1

Implementation notes

  • new route: POST /api/files/[id]/download-url
  • route uses session client (getClientFromSession) and files.createDownloadUrl
  • uses 30-second expiry and returns attributes.uri with fallback to deprecated downloadUrl
  • UI prevents row-click side effects when download action is clicked
  • UI shows an error snackbar when URL creation fails

Testing

  • yarn lint
  • yarn build

Risks/rollout

  • relies on browser navigation to presigned URL for download start behavior
  • if popup/new-tab is blocked, same-tab navigation fallback is used

https://vertexvis.atlassian.net/browse/PLAT-7588

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
vertex-dev-dashboard Ready Ready Preview, Comment Apr 7, 2026 8:33pm

Request Review

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 7, 2026

@danschultz danschultz changed the title PLAT-7588: Add file download action in dev dashboard [PLAT-7588] Add file download action in dev dashboard Apr 7, 2026
@danschultz danschultz merged commit 8882088 into main Apr 8, 2026
4 checks passed
@danschultz danschultz deleted the codex-plat-7588-file-download branch April 8, 2026 15:33
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