Skip to content

PM-37319: Feat: Update premium dialog titles#7072

Merged
david-livefront merged 1 commit into
mainfrom
PM-37319-premium-dialog-title
Jun 17, 2026
Merged

PM-37319: Feat: Update premium dialog titles#7072
david-livefront merged 1 commit into
mainfrom
PM-37319-premium-dialog-title

Conversation

@david-livefront

@david-livefront david-livefront commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

🎟️ Tracking

PM-37319

📔 Objective

📸 Screenshots

Before After

@david-livefront david-livefront requested a review from a team as a code owner June 17, 2026 17:01
@david-livefront david-livefront added the ai-review-vnext Request a Claude code review using the vNext workflow label Jun 17, 2026
@github-actions github-actions Bot added app:password-manager Bitwarden Password Manager app context app:authenticator Bitwarden Authenticator app context t:feature Change Type - Feature Development labels Jun 17, 2026
@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Reviewed the consolidation of premium-feature dialog titles to a single reusable premium_subscription_required string across 6 screens, plus the related refactor of the Send dialog states. The SendViewModel change (always showing FileTypeRequiresPremium instead of the isInAppUpgradeAvailable()-gated Error fallback) and the AddEditSendViewModel change (collapsing EmailAuthRequiresPremium and a defensive Error into a parameterized PremiumRequired(message) state) were traced and verified — both render correctly and remain consistent with the existing plan-modal navigation. Test updates match the code changes and the removed source strings have no remaining Kotlin references.

Code Review Details

No findings.

Notes (not blocking, not posted inline):

  • Three test names in AddEditSendScreenTest.kt still reference the removed EmailAuthRequiresPremium state; the bodies were correctly updated to PremiumRequired. Cosmetic only.

@david-livefront david-livefront force-pushed the PM-37319-premium-dialog-title branch from 6f63f35 to 9553161 Compare June 17, 2026 18:39
@david-livefront

Copy link
Copy Markdown
Collaborator Author

Thanks @SaintPatrck

@david-livefront david-livefront added this pull request to the merge queue Jun 17, 2026
Merged via the queue into main with commit 5b8dd94 Jun 17, 2026
25 checks passed
@david-livefront david-livefront deleted the PM-37319-premium-dialog-title branch June 17, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review-vnext Request a Claude code review using the vNext workflow app:authenticator Bitwarden Authenticator app context app:password-manager Bitwarden Password Manager app context t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants