-
Notifications
You must be signed in to change notification settings - Fork 85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GSoC 2024: GitHub Discussion UI at Home, Discussion and Contributor's route #463
GSoC 2024: GitHub Discussion UI at Home, Discussion and Contributor's route #463
Conversation
… type.ts and gh_events.ts
… type.ts and gh_events.ts
…4/leaderboard into refactor-scrapper-gsoc
…4/leaderboard into refactor-scrapper-gsoc
… as requested in review
…o discussion dir present
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@dgparmar14 Builds failing when there is no discussion directory (which happens when no discussions were scraped so far and none were present to be scrapped in the last 24 hours or the given scraping time range) |
Test: Refactor github scraper and integrate github discussions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes: LGTM ✅
Tested discussions locally and workflow in https://github.com/coronasafe/leaderboard-data-test/actions/runs/10089604078. Working as expected! ✅
Nice work @dgparmar14
Pull Request Description
Overview
This PR introduces a comprehensive update to display GitHub Discussions across various routes: Home, Discussions, and Profile. It also adds functionality to filter discussions based on their category and creation date. Additionally, discussions will be integrated with activities on the contributor's route.
Fixes - #212
Changes
New Routes and Components
New Discussions Route:
New Components:
Library Update
fetchGithubDiscussionForUser()
fetchGithubDiscussion()
: Fetches a specified number of discussions. If no number is provided, it returns all discussions.Discussions UI
Desktop View
Untitled.video.-.Made.with.Clipchamp.mp4
Mobile View
Untitled.video.-.Made.with.Clipchamp.2.mp4
1. Discussions Route:
2. Home Route:
3. Contributors Profile Route:
My Tasks
Mid Term Evaluation PR : Refactor Scraper