Skip to content

feat: add eliminated teams tracking to leaderboard, optimize data fetching, and configure Netlify build settings - #1

Merged
AdhwaithAS merged 2 commits into
gtech-mulearn:mainfrom
AdhwaithAS:main
Jul 23, 2026
Merged

feat: add eliminated teams tracking to leaderboard, optimize data fetching, and configure Netlify build settings#1
AdhwaithAS merged 2 commits into
gtech-mulearn:mainfrom
AdhwaithAS:main

Conversation

@AdhwaithAS

Copy link
Copy Markdown
Collaborator

No description provided.

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

@AdhwaithAS is attempting to deploy a commit to the MuLearn Team on Vercel.

A member of the Team first needs to authorize it.

@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds eliminated-team tracking and updates several frontend, API, caching, and deployment paths. The main changes are:

  • Marks eliminated teams on leaderboards and excludes them from registration.
  • Adds avatar uploads to the profile editor and expands accepted image formats.
  • Removes the prediction requirement from profile-task verification.
  • Adds visibility-aware leaderboard polling and response caching.
  • Configures Netlify builds for the Next.js frontend.

Confidence Score: 5/5

This looks safe to merge.

  • No new blocking issue distinct from the existing findings was found in the updated code.

Important Files Changed

Filename Overview
frontend/utils/constants.js Adds shared eliminated-team and available-team lists.
frontend/app/api/v1/register/route.js Uses the available-team list for server-side registration validation.
frontend/app/profile/[id]/components/useProfileState.js Updates avatar validation, input reset behavior, and modal integration.
frontend/utils/football-api.js Adds five-minute revalidation to match-data fetches.
frontend/app/api/v1/live-stats/route.js Removes referral analytics and adds shared-cache response headers.
netlify.toml Configures root-level Netlify builds for the frontend.

Reviews (2): Last reviewed commit: "chore: disable secrets scanning in Netli..." | Re-trigger Greptile

Comment on lines +308 to +309
if (file.size > 5 * 1024 * 1024) {
setUploadError("Image must be under 5MB.");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Avatar Size Limits Conflict

Images between 3 MB and 5 MB pass this new client check, but the unchanged API limit rejects them at 3 MB. The modal advertises 5 MB, so users can select a supposedly valid image only to have the upload fail.

Suggested change
if (file.size > 5 * 1024 * 1024) {
setUploadError("Image must be under 5MB.");
if (file.size > 3 * 1024 * 1024) {
setUploadError("Image must be under 3MB.");

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Comment thread frontend/utils/football-api.js
@AdhwaithAS
AdhwaithAS merged commit 64aa2c6 into gtech-mulearn:main Jul 23, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant