fix: validate inverted reporting date range on contributor profile - #175
fix: validate inverted reporting date range on contributor profile#175aryanorastar wants to merge 1 commit into
Conversation
Show a clear error when start date is after end date instead of treating the inverted window as an empty contribution period.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review. WalkthroughThe contributor profile now validates reporting date ranges. Reversed ranges clear filtered contributions, disable export, constrain date inputs, show an alert, use placeholder metrics, and display a shared message in pull-request and issue tables. ChangesContributor reporting validation
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This PR adds validation for inverted contributor reporting date ranges and prevents them from appearing as real empty periods. No actionable merge-blocking risk remains beyond normal checks and review. Possibly related PRs
Suggested labels: Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 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 |
Summary
Fixes #174
On the Contributor Profile reporting window, choosing a start date after the end date previously filtered contributions to zero with no validation message — easy to misread as “no activity.”
This change:
YYYY-MM-DDrangesInvalid date range: Start date cannot be later than end date.—, table empty copy uses the same validation message)min/maxhints on the date inputsVerification
npm test -- --run— 44/44 passed (includes newreportingDateRangeunit tests)2026-08-20/2026-08-10→ invalid; equal or ordered dates → validHonest gaps
Summary by CodeRabbit
Bug Fixes
Tests