De-duplicate lists: dashboard Home, analytics-only Insights, enriched…#12
Conversation
… stats One job per tab (unanimous roundtable + roadmap): - Home is now a dashboard: hero + Upcoming rail + a wired "Your subscriptions (N active / M total)" row into the Subscriptions tab. Dropped the full list (it duplicated the Subscriptions tab); converted the screen from FlatList to ScrollView. - Insights is analytics-only: removed the "History" list (a straight copy of the Subscriptions tab). Also reconciles counts (Paying / On trial / Paused / Cancelled now sum to the total), elevates savings to a hero "Cut from cancellations" card, discloses trial subs in the spend total, and shows total tracked in the header. - ListHeading action is now optional, removing the dead "View All" from every header; ready to carry a real wired action later (e.g. Home Upcoming -> future Calendar view). The Subscriptions tab is now the single owner of the full list. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (4)
Summary by CodeRabbit
WalkthroughThe dashboard updates home subscription presentation, expands Insights portfolio metrics and savings displays, removes Insights history cards, and makes ChangesSubscription dashboard
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 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 |
… stats
One job per tab (unanimous roundtable + roadmap):
The Subscriptions tab is now the single owner of the full list.