Remove contributions component and routes/pages#1035
Remove contributions component and routes/pages#1035Lakshya-2440 wants to merge 1 commit intoalphaonelabs:mainfrom
Conversation
👀 Peer Review RequiredHi @Lakshya-2440! This pull request does not yet have a peer review. Before this PR can be merged, please request a review from one of your peers:
Thank you for contributing! 🎉 |
|
Note
|
| Cohort / File(s) | Summary |
|---|---|
Contributor Templates web/templates/web/contributor_detail.html, web/templates/web/contributors_list.html |
Removed contributor detail page template (414 lines) with charts, metrics grid, and timeline; removed contributors list template (251 lines) with ranking, statistics cards, and contributor grid. |
URL Routing web/urls.py |
Removed two URL patterns: contributors list view and contributor detail view endpoints. |
View Functions and Constants web/views.py |
Removed GITHUB_REPO and GITHUB_API_BASE constants; removed get_user_contribution_metrics(), contributor_detail_view(), and contributors_list_view() functions that handled GitHub API queries and contributor aggregation. |
Homepage Layout web/templates/index.html |
Removed "View Contributions" card block from section 6; updated grid from two-column to single-column layout. |
Estimated code review effort
🎯 3 (Moderate) | ⏱️ ~22 minutes
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | The title accurately summarizes the main purpose of the changeset: removing the contributions component and its associated routes and pages across multiple files. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
📝 Coding Plan
- Generate coding plan for human review comments
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands and usage tips.
Tip
You can disable the changed files summary in the walkthrough.
Disable the reviews.changed_files_summary setting to disable the changed files summary in the walkthrough.
There was a problem hiding this comment.
Pull request overview
This PR removes the GitHub “contributions/contributors” feature from the Django web app by deleting its views, routes, and templates, and updating the homepage to no longer link to it.
Changes:
- Removed contributor list/detail views and related GitHub API/GraphQL logic from
web/views.py - Removed contributor routes from
web/urls.py - Deleted contributor templates and removed the “View Contributions” CTA from the homepage
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
web/views.py |
Deletes contributor-related view logic; leaves behind an unused GitHub API helper to clean up. |
web/urls.py |
Removes /contributors/ and /contributors/<username>/ routes. |
web/templates/web/contributors_list.html |
Deletes contributors list page template. |
web/templates/web/contributor_detail.html |
Deletes contributor detail page template. |
web/templates/index.html |
Removes “View Contributions” card and adjusts the section layout accordingly. |
Checklist
Purpose
Remove the contributions component and its associated routes, pages, views, and templates from the codebase.
Key Changes
Frontend Templates:
Backend Routing & Views:
contributors/(contributors_list_view)contributors/<str:username>/(contributor_detail_view)contributors_list_view()- aggregated contributors by GitHub PR activitycontributor_detail_view()- rendered detailed GitHub contributor statsget_user_contribution_metrics()- performed GitHub GraphQL queries for contributor dataGITHUB_REPOandGITHUB_API_BASEImpact