Skip to content

Conversation

Adez017
Copy link
Contributor

@Adez017 Adez017 commented Sep 15, 2025

Description

This Pr includes the addition of the dashboard in the place of the contributors section in the dashboard

Fixes #158

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

Added the leaderboard implementtation in a separate file

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/devices
  • My changes do not generate new console warnings or errors , I ran npm run build and attached scrrenshot in this PR.
  • This is already assigned Issue to me, not an unassigned issue.

Copy link

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 👇🏻

  1. Get free Consultation use code recode50 to get free: Mentorship for free.

  2. Get the Ebook for free use code recode at checkout: Data Science cheatsheet for Beginners.

  3. 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! 😊

Copy link

vercel bot commented Sep 15, 2025

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

A member of the Team first needs to authorize it.

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 15, 2025

Hi @sanjay-kv @iitzIrFan , to make things clear this is the implementation for the leaderboard .

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 15, 2025

The only issue persists is :

  1. the giveaway cards are misaligned.
image
  1. Some UI need to be fixed for better UX:
image
  1. on clicking the giveaway the sidebar changed :
image

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 15, 2025

Also to be note down, i had implemented the caching but , for the initial load it is taking time as we are fetching data across the 9 or 10 repos, i think @sanjay-kv @itz

@iitzIrFan
Copy link
Member

image

If this error is due to github access token then, it should be fine when @sanjay-kv merges it into main.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a leaderboard feature to replace the existing contributors section in the dashboard. The main purpose is to enhance community engagement by providing real-time GitHub-based contributor rankings with an improved user interface.

Key changes include:

  • Replaced hardcoded contributor data with dynamic GitHub API integration
  • Added a new standalone LeaderBoard component with filtering and pagination
  • Enhanced the stats provider to fetch real contributor data from all organization repositories

Reviewed Changes

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

Show a summary per file
File Description
src/pages/dashboard/index.tsx Major refactoring to remove hardcoded leaderboard logic and integrate new LeaderBoard component
src/lib/statsProvider.tsx Enhanced to fetch real contributor data from GitHub API with proper token authentication
src/components/navbar/NavbarIcon.tsx Modified to support both dashboard and navbar usage patterns through type guards
src/components/dashboard/LeaderBoard/leaderboard.tsx New comprehensive leaderboard component with search, pagination, and responsive design
src/components/dashboard/LeaderBoard/leaderboard.css Complete styling for the new leaderboard component with dark/light theme support
package.json Updated dotenv dependency version
docusaurus.config.ts Added environment variable configuration for GitHub token access

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

Comment on lines +458 to +459
text="LeaderBoard
"
Copy link
Preview

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

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

There's an unnecessary line break in the text string. Should be 'LeaderBoard' on a single line.

Suggested change
text="LeaderBoard
"
text="LeaderBoard"

Copilot uses AI. Check for mistakes.

@@ -1,64 +1,201 @@
// src/lib/statsProvider.tsx
Copy link
Preview

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

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

The file path comment should match the actual file path: src/lib/statsProvider.tsx

Copilot uses AI. Check for mistakes.

@@ -0,0 +1,319 @@
// src/pages/dashboard/LeaderBoard/leaderboard.tsx
Copy link
Preview

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

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

The file path comment is incorrect. Should be src/components/dashboard/LeaderBoard/leaderboard.tsx to match the actual file location.

Suggested change
// src/pages/dashboard/LeaderBoard/leaderboard.tsx
// src/components/dashboard/LeaderBoard/leaderboard.tsx

Copilot uses AI. Check for mistakes.

@iitzIrFan iitzIrFan added the under review Review under the maintainers or the admins label Sep 15, 2025
@iitzIrFan iitzIrFan moved this to In Progress in @recode-web Sep 15, 2025
Copy link

vercel bot commented Sep 15, 2025

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

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

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 15, 2025

image If this error is due to github access token then, it should be fine when @sanjay-kv merges it into main.

Hi @iitzIrFan , no it will load but will take some time like a minute or less

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 15, 2025

Did you see the others changes @iitzIrFan ?

@iitzIrFan
Copy link
Member

@Adez017 Yeah it looks fine !

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 15, 2025

@Adez017 Yeah it looks fine !

Check out the others tabs also like giveaway , etc . i think we need to fix the cards on the giveaway

@iitzIrFan
Copy link
Member

@Adez017 The sidebar and the body needs some changes, you can work on that, if you like. You can raise issue for it & work on it !

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 15, 2025

@Adez017 The sidebar and the body needs some changes, you can work on that, if you like. You can raise issue for it & work on it !

could you help i out , i had tried but one thing got fix other breaks out . i will try from my end but need your support thouroughly

@iitzIrFan
Copy link
Member

@Adez017 The sidebar and the body needs some changes, you can work on that, if you like. You can raise issue for it & work on it !

could you help i out , i had tried but one thing got fix other breaks out . i will try from my end but need your support thouroughly

Sure, just a small request for this PR can you optimize the loading behaviour as it is still not loading on my side from past 5-7 mins !

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 15, 2025

@Adez017 The sidebar and the body needs some changes, you can work on that, if you like. You can raise issue for it & work on it !

could you help i out , i had tried but one thing got fix other breaks out . i will try from my end but need your support thouroughly

Sure, just a small request for this PR can you optimize the loading behaviour as it is still not loading on my side from past 5-7 mins !

let me check , will update soon @iitzIrFan

Copy link
Member

@iitzIrFan iitzIrFan left a comment

Choose a reason for hiding this comment

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

Changes to optimize the data loading behaviour assigned to @Adez017 rest is fine.

@sanjay-kv

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 16, 2025

Hi @iitzIrFan , i had pushed changes for data loading optimization . let me know if it works or not .
also at my the total time taken from running npm start to data shown is around 49 secs.
CC: @sanjay-kv

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 16, 2025

Also, i want you to please check out the giveaway page, I think it needs a little fix @iitzIrFan

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 16, 2025

Also @iitzIrFan , this need to be fixed . we need the previous and next button here .
image

@Adez017
Copy link
Contributor Author

Adez017 commented Sep 16, 2025

@sanjay-kv , waiting for you in the meet

@sanjay-kv
Copy link
Member

@sanjay-kv , waiting for you in the meet

joining in

@sanjay-kv sanjay-kv merged commit 047652a into recodehive:main Sep 16, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in @recode-web Sep 16, 2025
@Adez017
Copy link
Contributor Author

Adez017 commented Sep 17, 2025

@sanjay-kv , add the lables over here also . and if possible could you give level 3 here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc25 level 3 under review Review under the maintainers or the admins
Projects
Status: Done
Status: Q4 2025 – Oct-Dec
Development

Successfully merging this pull request may close these issues.

The Leaderboard
4 participants