Skip to content

Fix: Subscriber Growth#50

Merged
dcodesdev merged 10 commits intomainfrom
fix/subs-growth
Dec 26, 2025
Merged

Fix: Subscriber Growth#50
dcodesdev merged 10 commits intomainfrom
fix/subs-growth

Conversation

@dcodesdev
Copy link
Copy Markdown
Owner

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 26, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Review Updated (UTC)
letter-space-docs Ignored Ignored Preview Dec 26, 2025 10:32am
letter-space-landing-page Ignored Ignored Preview Dec 26, 2025 10:32am

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 26, 2025

Caution

Review failed

The pull request is closed.

Summary by CodeRabbit

  • Bug Fixes

    • Fixed dashboard subscriber growth calculation to accurately track new subscriber acquisition over the 6-month reporting period.
  • Chores

    • Updated pnpm package manager to version 10.26.2 across all environments.
    • Enhanced development documentation and added automated release versioning script.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

This PR adds developer guidance documentation (CLAUDE.md files for root and apps), updates the pnpm package manager from version 10.10.0 to 10.26.2 across Dockerfiles and root configuration, introduces a release script for version bumping and Git tagging, distributes historical seed data over a 2-year window for realistic testing, and refactors the dashboard subscriber growth calculation to compute baseline subscriber counts and adjust filtering logic for campaign statistics.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • PR #15: Modifies the same apps/backend/src/dashboard/query.ts file—main PR adds baselineSubscriberCount calculation and adjusts messageStats grouping, while PR #15 adjusts deliveryRate calculation in the same query handler.
  • PR #17: Updates pnpm versions in Dockerfile and Dockerfile.node—both PRs directly modify these files for the same version management changes.
  • PR #13: Both PRs modify subscriber growth feature—main PR updates backend dashboard query with baseline handling logic, while PR #13 modifies subscriberGrowthQuery SQL granularity and related UI labeling.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/subs-growth

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 45ad2c7 and 90381f1.

📒 Files selected for processing (11)
  • CLAUDE.md
  • Dockerfile
  • Dockerfile.node
  • RELEASE_NOTES.md
  • apps/backend/CLAUDE.md
  • apps/backend/package.json
  • apps/backend/prisma/seed.ts
  • apps/backend/src/dashboard/query.ts
  • apps/web/CLAUDE.md
  • package.json
  • scripts/release.sh

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 and usage tips.

@dcodesdev dcodesdev merged commit 0ba3ec0 into main Dec 26, 2025
3 of 4 checks passed
@dcodesdev dcodesdev deleted the fix/subs-growth branch December 26, 2025 10:32
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