fix(desktop): generic monthly-limit popup copy for menu-bar trigger#7533
Draft
mdmohsin7 wants to merge 2 commits into
Draft
fix(desktop): generic monthly-limit popup copy for menu-bar trigger#7533mdmohsin7 wants to merge 2 commits into
mdmohsin7 wants to merge 2 commits into
Conversation
The menu-bar Screen Capture / Audio Recording toggles in OmiApp.swift post showUsageLimitPopup with reason "trial_expired", which fell to the default branch in UsageLimitPopupView and returned the same copy as the transcription-cap case: "Upgrade to make sure your new recordings aren't lost." That phrasing reads as a data-loss threat. For users tripping a stale desktop_isPaywalled flag (now self-healed at launch by #7517) while also at their Neo chat cap, listening was never actually paywalled server-side; the wording made them think their recordings were at risk when only the chat cap was hit. Switch the default branch to plan-agnostic copy: "Upgrade to keep using Omi without restrictions." The transcription-specific case is unchanged (real transcription cap → recordings copy is accurate).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
OmiApp.swift:1033, 1070postshowUsageLimitPopupwithreason: "trial_expired". That string fell through to thedefaultbranch ofUsageLimitPopupView.body_textand returned the same copy as the transcription-cap case: "Upgrade to make sure your new recordings aren't lost."desktop_isPaywalledflag (now self-healed at launch by desktop: clear sticky paywalled flag on launch for paid-plan users (follow-up to #7514) #7517) while also at their Neo chat cap, listening was never actually paywalled server-side — but the copy made them believe their recordings were at risk.defaultbranch to plan-agnostic copy: "Upgrade to keep using Omi without restrictions." The explicit"transcription"case is unchanged (real transcription-cap → recordings copy is accurate).Why now
Surfaced while investigating the same Neo grandfathered cohort #7517 fixed at launch. Several users on legacy Unlimited / Neo plans were seeing the recordings-loss copy on the menu-bar toggle path while their backend state correctly reported
trial_expired=falseandchat_quota_allowed=true. After #7517 self-heals the stale flag on next launch, this popup should no longer fire for them in normal flow — but the copy bug also affects any legitimate"trial_expired"paywall (basic trial expired with no BYOK), where the same data-loss framing is inaccurate.Test plan
isPaywalled=true→ popup body reads "Upgrade to keep using Omi without restrictions."🤖 Generated with Claude Code