fix(seed): revenue chart should not spike on the current month#37
Merged
Conversation
…spike The paid-invoice month weighting front-loaded the current month, so the dashboard Monthly Revenue chart showed the (partial) current month at ~2.5x every prior full month — reading as 'seed dumped recent data' rather than a real business. Rebalanced so paid invoices spread across the trailing 6 months with only a partial current month.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the demoSeed function in seed.ts to adjust the distribution of paid invoices in the PAID_MONTHS array. The change spreads invoices more evenly across the trailing six months to create a more realistic revenue chart for demo data, avoiding a spike in the final month. I have no feedback to provide as there are no review comments.
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
1 participant
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.
Why
Live dashboard Monthly Revenue showed the in-progress current month at ~$98k vs ~$10–38k for every prior full month — a 2.5× artifact from the paid-invoice month weighting front-loading month 0. That reads as synthetic seed data, not a real business.
What
Rebalanced
PAID_MONTHSso paid invoices spread fairly evenly across the trailing 6 months with only a partial current month.Verification
Local fresh reseed: per-month paid totals are now in a comparable range with the current month mid-range (no final-month spike); the curve is naturally bumpy (realistic for project-based billing).
tsc/eslintclean. Before/after chart screenshots in the final report.