Skip to content

fix(#246): replace dashboard analytics fetch calls with authFetch#420

Open
0xPyxis wants to merge 1 commit into
MRIARC-08:mainfrom
0xPyxis:main
Open

fix(#246): replace dashboard analytics fetch calls with authFetch#420
0xPyxis wants to merge 1 commit into
MRIARC-08:mainfrom
0xPyxis:main

Conversation

@0xPyxis

@0xPyxis 0xPyxis commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Closes #246

Replaced direct fetch() calls in dashboard analytics components with authFetch() so authentication refresh handling is applied consistently.

CHANGES

  • Updated ProgressDashboard to use authFetch for /api/analytics/overview
  • Updated StreakDashboard to use authFetch for /api/analytics/streak
  • Preserved existing loading, error, and abort-signal behavior

@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

@0xPyxis is attempting to deploy a commit to the Adarsh's projects Team on Vercel.

A member of the Team first needs to authorize it.

@MRIARC-08 MRIARC-08 added gssoc gssoc level:beginner GSSoC difficulty: beginner type:bug Bug fix contribution labels Jun 25, 2026
@MRIARC-08

Copy link
Copy Markdown
Owner

@0xPyxis Code-side checks look good, but I’m holding this for maintainer manual UI review because it touches visible dashboard components.

Local checks:

  • pnpm exec eslint src/components/ProgressDashboard.tsx src/components/StreakDashboard.tsx passes.
  • pnpm exec tsc --noEmit passes.
  • pnpm test passes: 8 files / 35 tests.

@MRIARC-08 Please manually verify the dashboard progress/streak loading and error states before merge.

@0xPyxis

0xPyxis commented Jun 28, 2026

Copy link
Copy Markdown
Contributor Author

hey could you review this ??

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

Labels

gssoc gssoc level:beginner GSSoC difficulty: beginner type:bug Bug fix contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Dashboard analytics components bypass authFetch refresh handling

2 participants