Skip to content

feat(files): add opt-in encrypted PDF thumbnail uploads (WPB-29908) - #23050

Open
cristianoliveira wants to merge 4 commits into
mainfrom
WPB-29908-pdf-preview-upload
Open

cristianoliveira wants to merge 4 commits into
mainfrom
WPB-29908-pdf-preview-upload

Conversation

@cristianoliveira

@cristianoliveira cristianoliveira commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor
TaskWPB-29908 [Web] E2e PDF preview on chat

Problem

Encrypted chat assets carry the original PDF but no first-page thumbnail, so the receiver cannot show a preview without downloading the document.

Solution

  • Render a bounded first-page PNG in the browser.
  • Upload it through the existing encrypted asset service with an independent key and the original asset's retention policy.
  • Encode its descriptor in the existing optional Asset.preview field and preserve it through message mapping.
  • Preserve original-file sending when optional preview generation/upload fails, and cover cancellation and cleanup paths.
  • Register e2e-pdf-preview but keep it off by default in every build, including Edge, until the receiver UI lands. Explicit opt-in remains available.

No backend, protocol-schema, or dependency changes. This PR adds no receiver preview UI.

Reference

https://wearezeta.atlassian.net/browse/WPB-29908

@cristianoliveira cristianoliveira added the semver:minor Backward-compatible WebApp functionality. Next release increments MINOR label Nov 11, 2026
@cristianoliveira cristianoliveira changed the title feat(conversation): add opt-in encrypted PDF thumbnail uploads (WPB-29908) feat(files): add opt-in encrypted PDF thumbnail uploads (WPB-29908) Nov 11, 2026
arjita-mitra
arjita-mitra previously approved these changes Nov 11, 2026
@cristianoliveira
cristianoliveira force-pushed the WPB-29908-pdf-preview-upload branch from b0cf359 to 30c437f Compare November 11, 2026 13:46
@arjita-mitra
arjita-mitra force-pushed the WPB-29908-pdf-preview-upload branch from 7fb78a4 to 65237a8 Compare November 12, 2026 08:44
Base automatically changed from WPB-29908-shared-file-preview to main November 12, 2026 10:01
@cristianoliveira
cristianoliveira force-pushed the WPB-29908-pdf-preview-upload branch from 65237a8 to 4681651 Compare November 12, 2026 10:01

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:minor Backward-compatible WebApp functionality. Next release increments MINOR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants