-
-
Notifications
You must be signed in to change notification settings - Fork 313
Fix hackathon page view stats and add timeframe-based metrics #4813
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
Co-authored-by: DonnieBLT <[email protected]>
|
@copilot update the view count to reflect the timeframe of the hackathon please also show an additional stat for all time views and make sure they are labeled correctly |
Co-authored-by: DonnieBLT <[email protected]>
Updated the page view stats to show hackathon timeframe-based views and all-time views with clear labels (commit 28e53dc). Changes:
Example display: The hackathon views (40) now correctly represent views during the hackathon period, while all-time views (64) show the complete history. Both metrics are clearly labeled to avoid confusion. |
Hackathon page view counts were inconsistent: the stats card showed 40 views while the bottom-left widget showed 64 views for the same page.
Root Cause
Two different query patterns were aggregating page views:
The stats card used exact path matching over 14 days, while the widget used substring matching over 30 days.
Changes
Initial Fix
path=) to substring match (path__contains=) to match widget behaviorEnhanced Metrics (Based on Feedback)
The stats card now displays hackathon-specific metrics with proper context, and the sparkline chart reflects activity during the hackathon period.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.