Skip to content

De-duplicate lists: dashboard Home, analytics-only Insights, enriched…#12

Merged
saikumarbt merged 1 commit into
mainfrom
insights-home-dashboard
Jul 15, 2026
Merged

De-duplicate lists: dashboard Home, analytics-only Insights, enriched…#12
saikumarbt merged 1 commit into
mainfrom
insights-home-dashboard

Conversation

@saikumarbt

Copy link
Copy Markdown
Owner

… 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.

… 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>
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4189c18f-6bd8-4874-8295-78de523a0700

📥 Commits

Reviewing files that changed from the base of the PR and between 697603e and 5bf880f.

📒 Files selected for processing (4)
  • app/(tabs)/index.tsx
  • components/Insights.tsx
  • components/ListHeading.tsx
  • type.d.ts

Summary by CodeRabbit

  • New Features

    • Added a subscriptions overview card showing active and total subscription counts.
    • Added clearer portfolio insights for paying, trial, paused, and cancelled subscriptions.
    • Added yearly savings estimates from cancelled subscriptions.
    • Added configurable section actions that can be shown or hidden as needed.
  • Bug Fixes

    • Added an empty state when no subscriptions are available.
    • Improved navigation from the subscriptions overview to the full subscriptions list.
    • Refined Insights labels and chart descriptions for improved clarity.

Walkthrough

The dashboard updates home subscription presentation, expands Insights portfolio metrics and savings displays, removes Insights history cards, and makes ListHeading trailing actions configurable.

Changes

Subscription dashboard

Layer / File(s) Summary
Configurable heading actions
type.d.ts, components/ListHeading.tsx
ListHeading now accepts optional action text and callbacks, rendering the action control only when both are provided.
Home subscription summary
app/(tabs)/index.tsx
The home screen uses a ScrollView and replaces individual subscription cards with an empty state or a pressable summary card linking to /subscriptions.
Insights portfolio metrics
components/Insights.tsx
Insights adds status counts and yearly savings, updates portfolio tiles and cancellation savings display, and removes subscription history cards.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Poem

A bunny counts subscriptions bright,
With paused and trialled tiles in sight.
Cancellations save a yearly treat,
Home cards hop to lists so neat.
Configurable headings make it sweet! 🐇

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title matches the main theme of de-duplicating lists across Home and Insights, though it is slightly truncated.
Description check ✅ Passed The description directly explains the Home, Insights, and ListHeading changes and aligns with the PR objectives.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch insights-home-dashboard

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@saikumarbt
saikumarbt merged commit e5ecdac into main Jul 15, 2026
1 check passed
@coderabbitai coderabbitai Bot mentioned this pull request Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant