Skip to content

[PM-33413] feat: Require premium for attachment preview - #2975

Draft
morganzellers-bw wants to merge 1 commit into
pm-33413-image-attachment-previewfrom
pm-33413-attachment-premium-gating
Draft

[PM-33413] feat: Require premium for attachment preview #2975
morganzellers-bw wants to merge 1 commit into
pm-33413-image-attachment-previewfrom
pm-33413-attachment-premium-gating

Conversation

@morganzellers-bw

Copy link
Copy Markdown
Contributor

Viewing and downloading existing attachments was previously ungated, unlike uploading new attachments. This adds the same Premium check to AttachmentPreviewHelper.showPreview, showing an upgrade alert instead of downloading when the active account lacks Premium.

🎟️ Tracking

📔 Objective

📸 Screenshots

@morganzellers-bw morganzellers-bw self-assigned this Aug 18, 2026
@github-actions github-actions Bot added app:password-manager Bitwarden Password Manager app context app:authenticator Bitwarden Authenticator app context t:feature labels Aug 18, 2026
…cription

Viewing and downloading existing attachments was previously ungated,
unlike uploading new attachments. This adds the same Premium check to
AttachmentPreviewHelper.showPreview, showing an upgrade alert instead
of downloading when the active account lacks Premium.
@morganzellers-bw
morganzellers-bw force-pushed the pm-33413-attachment-premium-gating branch from 8164890 to 835f873 Compare August 18, 2026 17:05
@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.42%. Comparing base (9ec4b95) to head (835f873).

Additional details and impacted files
@@                          Coverage Diff                          @@
##           pm-33413-image-attachment-preview    #2975      +/-   ##
=====================================================================
+ Coverage                              79.40%   79.42%   +0.02%     
=====================================================================
  Files                                   1175     1175              
  Lines                                  75413    75440      +27     
=====================================================================
+ Hits                                   59878    59920      +42     
+ Misses                                 15535    15520      -15     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

app:authenticator Bitwarden Authenticator app context app:password-manager Bitwarden Password Manager app context t:feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant