Skip to content

fix: paginate commit fetch in weekly-summary and hourly endpoints#896

Merged
Priyanshu-byte-coder merged 1 commit into
Priyanshu-byte-coder:mainfrom
Neal006:fix/weekly-summary-streak-pagination
May 24, 2026
Merged

fix: paginate commit fetch in weekly-summary and hourly endpoints#896
Priyanshu-byte-coder merged 1 commit into
Priyanshu-byte-coder:mainfrom
Neal006:fix/weekly-summary-streak-pagination

Conversation

@Neal006
Copy link
Copy Markdown
Contributor

@Neal006 Neal006 commented May 24, 2026

Summary

Brief description of what this PR does.

Closes #

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / code cleanup

Changes Made

  • ...
  • ...

How to Test

Steps for the reviewer to verify this works:

  1. ...
  2. ...

Screenshots (if UI change)

Checklist

  • Linked issue in summary
  • npm run lint passes locally
  • No TypeScript errors (npm run type-check)
  • Self-reviewed the diff
  • Added/updated tests if applicable

Copilot AI review requested due to automatic review settings May 24, 2026 04:57
@vercel
Copy link
Copy Markdown

vercel Bot commented May 24, 2026

@Neal006 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added gssoc26 GSSoC 2026 contribution type:bug GSSoC type bonus: bug fix labels May 24, 2026
@github-actions
Copy link
Copy Markdown

GSSoC Label Checklist 🏷️

@Priyanshu-byte-coder — please apply the appropriate labels before merging:

Difficulty (pick one):

  • level:beginner — 20 pts
  • level:intermediate — 35 pts
  • level:advanced — 55 pts
  • level:critical — 80 pts

Quality (optional):

  • quality:clean — ×1.2 multiplier
  • quality:exceptional — ×1.5 multiplier

Validation (required to score):

  • gssoc:approved — counts for points
  • gssoc:invalid / gssoc:spam / gssoc:ai-slop — does not score

Type labels (type:*) are auto-detected from files and title. Review and adjust if needed.
Points formula: (difficulty × quality_multiplier) + type_bonus

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your first PR on DevTrack! 🎉

A maintainer will review it within 48 hours. While you wait:

  • Make sure CI is passing (type-check + lint)
  • Double-check the PR description is filled out and the issue is linked
  • Feel free to ask questions in Discussions if you need help

If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors!

@Priyanshu-byte-coder
Copy link
Copy Markdown
Owner

The pagination logic is correct and needed. However, this PR still uses session.accessToken directly in both modified routes. PR #876 (security fix) removes accessToken from the session object — once that merges, these routes will return unauthorized for every request. Please update to use getGitHubAccessToken(req) from @/lib/server-github-token (same pattern being introduced in #876). Also note that weekly-summary/route.ts is also modified in #876, so you will need to rebase after that merges.

@Priyanshu-byte-coder Priyanshu-byte-coder added level:intermediate GSSoC: Intermediate difficulty (35 pts) gssoc:approved GSSoC: PR approved for scoring labels May 24, 2026
@Priyanshu-byte-coder Priyanshu-byte-coder merged commit 1c08827 into Priyanshu-byte-coder:main May 24, 2026
4 checks passed
@github-actions
Copy link
Copy Markdown

🎉 Merged! Thanks for contributing to DevTrack.

If the project has been useful to you, a ⭐ star on the repo is the easiest way to support it — it helps DevTrack get discovered by more developers.

Keep an eye on open issues for your next contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved GSSoC: PR approved for scoring gssoc26 GSSoC 2026 contribution level:intermediate GSSoC: Intermediate difficulty (35 pts) type:bug GSSoC type bonus: bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants