Added PR's to the leaderboard #584

Adez017 · 2025-09-22T16:08:10Z

Description

Fixes #532

Type of Change

New feature (e.g., new page, component, or functionality)
Bug fix (non-breaking change that fixes an issue)
UI/UX improvement (design, layout, or styling updates)
Performance optimization (e.g., code splitting, caching)
Documentation update (README, contribution guidelines, etc.)
Other (please specify):

Changes Made

Dependencies

List any new dependencies or tools required for this change.
Mention any version updates or configurations that need to be considered.

Checklist

My code follows the style guidelines of this project.
I have tested my changes across major browsers and devices
My changes do not generate new console warnings or errors .
I ran npm run build and attached screenshot(s) in this PR.
This is already assigned Issue to me, not an unassigned issue.

Co-authored-by: Adez017 <[email protected]>

Add clickable PR list modal to leaderboard for viewing contributor pull requests

github-actions · 2025-09-22T16:08:17Z

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs.

In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊

Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv.

We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰

🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨

📚Your perks for contribution to this community 👇🏻

Get free Consultation use code recode50 to get free: Mentorship for free.
Get the Ebook for free use code recode at checkout: Data Science cheatsheet for Beginners.
Check out this weekly Newsletter: Sanjay's Newsletter.

If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

vercel · 2025-09-22T16:08:30Z

@Adez017 is attempting to deploy a commit to the recode Team on Vercel.

A member of the Team first needs to authorize it.

Copilot

Pull Request Overview

This PR adds PR details functionality to the leaderboard by implementing a modal that displays detailed information about contributors' merged pull requests.

Added PR details tracking to contributors with title, URL, merge date, repository name, and PR number
Created a modal component to display a list of individual PRs when users click on PR badges
Implemented fallback mock data functionality for testing and error handling scenarios

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/lib/statsProvider.tsx	Extended data models and API processing to capture detailed PR information
src/components/dashboard/LeaderBoard/mockData.ts	Added mock contributor data with PR details for testing purposes
src/components/dashboard/LeaderBoard/leaderboard.tsx	Integrated modal functionality and clickable PR badges with fallback data handling
src/components/dashboard/LeaderBoard/leaderboard.css	Added comprehensive styling for the PR modal component
src/components/dashboard/LeaderBoard/PRListModal.tsx	New modal component for displaying detailed PR information

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-09-22T16:09:19Z

src/lib/statsProvider.tsx

  };
  merged_at?: string | null;
+  title?: string;
+  html_url?: string;


These optional properties in PullRequestItem interface duplicate existing properties. The interface already has html_url on line 64, and these additions create redundancy. Consider consolidating or clarifying the purpose of these duplicate fields.

Suggested change

html_url?: string;

pr_url?: string;

Copilot · 2025-09-22T16:09:20Z

src/lib/statsProvider.tsx

+
+          // Add detailed PR information
+          if (pr.title && pr.html_url && pr.merged_at && pr.number) {
+            contributor.prDetails = contributor.prDetails || [];


The prDetails array is already initialized on line 223 when creating a new contributor, making the null check on line 232 redundant. Consider removing this redundant initialization.

Suggested change

contributor.prDetails = contributor.prDetails || [];

src/components/dashboard/LeaderBoard/leaderboard.tsx

Co-authored-by: Copilot <[email protected]>

Copilot

Pull Request Overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

src/lib/statsProvider.tsx

Copilot · 2025-09-22T16:11:34Z

src/components/dashboard/LeaderBoard/leaderboard.tsx

+  // Use mock data only in development mode when there's an error or no contributors
+  const displayContributors =
+    (error || contributors.length === 0)
+      ? (typeof process !== "undefined" && process.env.NODE_ENV === "development"


The typeof process !== "undefined" check is unnecessary in a React/Node.js environment where process is always available. This check is typically used for browser environments, but here you're already checking process.env.NODE_ENV. Simplify to just check the environment directly.

Suggested change

? (typeof process !== "undefined" && process.env.NODE_ENV === "development"

? (process.env.NODE_ENV === "development"

Copilot · 2025-09-22T16:11:34Z

src/components/dashboard/LeaderBoard/leaderboard.tsx

+            {error && (
+              <div className="error-banner" style={{ 
+                padding: '12px', 
+                backgroundColor: isDark ? '#fee8e7' : '#fee8e7', 
+                color: '#dc2626', 
+                borderRadius: '8px', 
+                marginBottom: '16px',
+                fontSize: '14px',
+                textAlign: 'center'
+              }}>
+                Demo Mode: Showing sample data due to API configuration issue
+              </div>
+            )}


The backgroundColor is the same for both dark and light modes (#fee8e7). This will likely cause poor contrast in dark mode. The background should be different for dark mode, such as #991b1b or another dark-appropriate color.

Adez017 · 2025-09-22T16:17:47Z

here is the video demonstration : https://drive.google.com/file/d/1_vJSKd7sST_oI4KquAprXwCLcsKvUvk2/view?usp=sharing

vercel · 2025-09-23T12:28:21Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
recode-website	Ready	Preview	Comment	Sep 24, 2025 9:56am

Adez017 · 2025-09-23T12:28:38Z

@sanjay-kv wait for now , don't merge it , i need to sync with @iitzIrFan changes

Adez017 · 2025-09-23T12:33:49Z

Now ready to go @sanjay-kv , also @iitzIrFan , kindly review as of my end i test the filter also you created and its working but to be more sure you can test also

Copilot

Pull Request Overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

src/lib/statsProvider.tsx

src/components/dashboard/LeaderBoard/leaderboard.tsx

Co-authored-by: Copilot <[email protected]>

Copilot

Pull Request Overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

src/components/dashboard/LeaderBoard/leaderboard.tsx

iitzIrFan

It's amazingly done @Adez017 Thanks for this feature and yeah it is working with the filtering.
@sanjay-kv Ready for merge.

sanjay-kv · 2025-09-24T09:56:09Z

Love the collaboration here @iitzIrFan @Adez017

Adez017 · 2025-09-24T16:16:22Z

Hi @sanjay-kv , add the gssoc related labels

Copilot AI and others added 4 commits September 22, 2025 10:35

Initial plan

9684cf9

Implement PR list modal feature for leaderboard

588893b

Co-authored-by: Adez017 <[email protected]>

Add mock data for testing PR list modal functionality

5d5e4d5

Co-authored-by: Adez017 <[email protected]>

Added PR to the leaderboard

41d04bc

Add clickable PR list modal to leaderboard for viewing contributor pull requests

Copilot AI review requested due to automatic review settings September 22, 2025 16:08

Copilot AI reviewed Sep 22, 2025

View reviewed changes

Update src/components/dashboard/LeaderBoard/leaderboard.tsx

1affcc6

Co-authored-by: Copilot <[email protected]>

Copilot AI review requested due to automatic review settings September 22, 2025 16:10

Copilot AI reviewed Sep 22, 2025

View reviewed changes

Merge branch 'main' into Experiment

613ed70

sanjay-kv approved these changes Sep 23, 2025

View reviewed changes

sanjay-kv assigned Adez017 Sep 23, 2025

sanjay-kv added gssoc25 level 2 30 points labels Sep 23, 2025

sanjay-kv added this to @recode-web Sep 23, 2025

sanjay-kv moved this to In Progress in @recode-web Sep 23, 2025

sanjay-kv removed the status in @recode-web Sep 23, 2025

sanjay-kv added this to the recode:launch 3.0 milestone Sep 23, 2025

vercel bot had a problem deploying to Preview September 23, 2025 12:28 Failure

quick fix

5654470

Copilot AI review requested due to automatic review settings September 23, 2025 12:32

Copilot AI reviewed Sep 23, 2025

View reviewed changes

Update src/components/dashboard/LeaderBoard/leaderboard.tsx

a5d3961

Co-authored-by: Copilot <[email protected]>

Copilot AI review requested due to automatic review settings September 23, 2025 12:45

Copilot AI reviewed Sep 23, 2025

View reviewed changes

src/components/dashboard/LeaderBoard/leaderboard.tsx Show resolved Hide resolved

iitzIrFan moved this to In Progress in @recode-web Sep 23, 2025

iitzIrFan approved these changes Sep 23, 2025

View reviewed changes

iitzIrFan added Ready for Merge PR has passed reviews and checks, and is ready to be merged into the main branch. under review Review under the maintainers or the admins and removed under review Review under the maintainers or the admins labels Sep 23, 2025

vercel bot deployed to Preview September 24, 2025 09:56 View deployment

sanjay-kv merged commit a2401eb into recodehive:main Sep 24, 2025
3 checks passed

github-project-automation bot moved this from In Progress to Done in @recode-web Sep 24, 2025

	? (typeof process !== "undefined" && process.env.NODE_ENV === "development"
	? (process.env.NODE_ENV === "development"

Added PR's to the leaderboard #584

Added PR's to the leaderboard #584

Uh oh!

Conversation

Adez017 commented Sep 22, 2025

Description

Type of Change

Changes Made

Dependencies

Checklist

Uh oh!

github-actions bot commented Sep 22, 2025

Uh oh!

vercel bot commented Sep 22, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Sep 22, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 22, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI Sep 22, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 22, 2025

Choose a reason for hiding this comment

Uh oh!

Adez017 commented Sep 22, 2025

Uh oh!

vercel bot commented Sep 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Adez017 commented Sep 23, 2025

Uh oh!

Adez017 commented Sep 23, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

iitzIrFan left a comment

Choose a reason for hiding this comment

Uh oh!

sanjay-kv commented Sep 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Adez017 commented Sep 24, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

vercel bot commented Sep 23, 2025 •

edited

Loading

sanjay-kv commented Sep 24, 2025 •

edited

Loading