Closed sanjay-kv closed 6 days ago
I am willing to work on this issue!
much appreciated @Sid-80
@sanjay-kv We are having any api for getting data about PRs.
i think we should already have in leaderboard.js file
Hows this going @Sid-80
@sanjay-kv I am out of station!! I will surely complete it!!
@sanjay-kv In leaderboard.json, we are having only user data. Here we need project events!!
UI is ready, just data is remaining
@sanjay-kv UI
@Sid-80 what you suggest new api just for projects?
Yess. The api should return the above required data about projects. Or Modification in projects api to add the above additional details like last PR merged, open issues, open PRs!!
I think modification in the existing project API will do, additional API calling will make the loading data slower. Let me know whether you can do it or you need any help
We can display the above info after clicking on project details modal. What's your view
Performance will not be affected
then we can create new API and link to it. Sorry about the delay in communications.
Yepp!! No issues!! Patience is imp in Open Source
Please guide me to create api !!
@sanjay-kv I have to use any backend endpoint ??
I will connect with person who is good at api. So can collaborate with him.
Let me know!! Thank you!!
Hi @Sid-80,
I've created an API for the Projects Page, which allows you to retrieve all project details in a single API call. The data is automatically updated every hour, and the response time is super fast, so it won't slow down data loading.
If you want you use this API on the Projects Page. If you have any specific additional data you'd like to see in the API response, just let me know.
API Endpoint: https://gssoc-leaderboard.vercel.app/api/projects
Hii @devXprite, totalPullRequests are total PRs merged or number of PRs open ?
Hii @devXprite, totalPullRequests are total PRs merged or number of PRs open ?
They are Open PR. I think i should Rename it to openPullRequests
& openIssues
.
Yepp!! Better for understanding
Hii @devXprite, totalPullRequests are total PRs merged or number of PRs open ?
They are Open PR. I think i should Rename it to
openPullRequests
&openIssues
.
Yes, that would easy to understand for future enhancement
I will give it a try!! Will ping if any backend error occurs
Hii @devXprite, totalPullRequests are total PRs merged or number of PRs open ?
They are Open PR. I think i should Rename it to
openPullRequests
&openIssues
.Yes, that would easy to understand for future enhancement
Done :+1:. Now Changes are live.
@devXprite CORS issue
@sanjay-kv I think scrollbar doesn't look better on cards. What's your view ?
@devXprite Which year's projects will be returned, latest one?
@devXprite CORS issue
Fixed :+1:
@devXprite Which year's projects will be returned, latest one?
Yes, they are current year projects and are ordered by the most recent commit date.
@sanjay-kv I think scrollbar doesn't look better on cards. What's your view ?
yes, we can create a standard size without scrolls @devXprite is working on it. @Sid-80 I have another work would you like to take it?
Add more features like total open PR and issue to existing Project Page.