feat(purchase): mention Warp in Pro modal and suggest an account after purchase - #274
Conversation
…after purchase The Pro modal said Pro does not include Warp but not that Warp includes Pro, so some people bought both. It now describes Warp as a subscription that includes Pro while subscribed, with a link to Zaparoo Online. After a signed-out purchase, the modal offers an optional step to link Pro to a free Zaparoo Online account, which backs up the purchase and makes it usable across Android and iOS devices. It is skipped for signed-in users and for store-verified local recovery, where RevenueCat has no record to link. Warp checkout no longer lists Pro access as a benefit for people who already own lifetime Pro.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (13)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe Pro purchase modal adds post-purchase account linking, Warp information, and router navigation. Purchase recovery and signed-out purchases can show an account step. Warp checkout identifies separately owned lifetime Pro access. Localized purchase text and tests were updated. ChangesPro purchase and account linking
Warp ownership display
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Feature Sequence Diagram(s)sequenceDiagram
participant Customer
participant ProPurchaseModal
participant purchasePackage
participant Router
Customer->>ProPurchaseModal: Start Pro purchase
ProPurchaseModal->>purchasePackage: Submit purchase
purchasePackage-->>ProPurchaseModal: Return purchase result
ProPurchaseModal->>Customer: Show account step when signed out
Customer->>ProPurchaseModal: Select sign in
ProPurchaseModal->>Router: Navigate to /settings/online
Merge Risk: ⚪ Minimal · up to The updated purchase flow can direct signed-out purchasers to account sign-in, where their purchase identity is linked for cross-device access. No merge-blocking defect was identified. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 4 files. (9 skipped: 9 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
scan.purchaseAccount*strings are now wired up, with updated copy and new keys translated in all locales.Summary by CodeRabbit
New Features
Documentation