-
-
Notifications
You must be signed in to change notification settings - Fork 313
Revert "Updates dec14" #5283
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
Revert "Updates dec14" #5283
Conversation
❌ Console Statements DetectedFound 1 file(s) with Why remove console statements?
Files with violations:website/static/js/repo_detail.js: How to fix:
Note:
Please remove all console statements and push your changes. |
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.
Pull request overview
This is a revert of PR #5281, which rolls back a large set of feature additions and changes made in the December 14th update. The revert removes various enhancements including improved repository data management, organization features, UI updates, and GitHub API integrations.
Key changes:
- Removal of enhanced repository and organization GitHub data refresh functionality
- Revert of UI/UX improvements to templates and styling
- Removal of additional context fields and data serialization in views
- Restoration of previous error handling and pagination settings
Reviewed changes
Copilot reviewed 26 out of 27 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| website/views/repo.py | Removed enhanced error messages, organization context data, and GitHub issue serialization from repository refresh endpoint |
| website/views/project.py | Removed Tag import, restored stargazers auto-fetching, removed technical details refresh, and reverted related repos feature |
| website/views/organization.py | Removed organization repository refresh API endpoint, list mode view, sorting features, and ContentType imports |
| website/views/core.py | Removed cron schedule parsing functions and frequency-based sorting for management commands |
| website/templates/status_page.html | Added management commands link |
| website/templates/repo/repo_list.html | Removed breadcrumbs and simplified repository card display |
| website/templates/projects/repo_detail.html | Extensive layout revert including stargazers section, GitHub issues display, and technical details sections |
| website/templates/organization/organization_list_mode.html | Entire file removed |
| website/templates/organization/organization_list.html | Removed GSOC tag styling, list mode link, sorting dropdown, top repos display, and refresh button |
| website/templates/organization/organization_detail.html | Removed repositories section and refresh functionality |
| website/templates/map.html | Added attribution comment |
| website/templates/management_commands.html | Removed frequency column, file info sorting, and improved dark mode styling |
| website/templates/includes/sidenav.html | Removed pinning functionality and conditional organization menu display |
| website/templates/includes/navbar.html | Removed conditional organization menu items |
| website/templates/includes/header.html | Removed conditional organization menu items and pinned sidebar logic |
| website/static/js/repo_detail.js | Removed GitHub refresh functionality, tag updates, and technical field updates; restored console.error statements |
| website/static/js/organization_list.js | Entire file removed |
| website/static/css/custom-scrollbar.css | Changed from hidden scrollbar to visible styled scrollbar |
| website/management/commands/run_daily.py | Added cron_send_reminders command call |
| website/management/commands/populate_github_org.py | Entire file removed |
| run.sh | Removed migrations, browser opening logic, and poetry run prefix |
| project_channels.csv | New file added with 114 Slack channel mappings |
| blt/urls.py | Removed organization list mode view and refresh API endpoint |
| blt/settings.py | Removed DEBUG conditional for email backend |
| blt/middleware/throttling.py | Removed DEBUG bypass for throttling |
| .pre-commit-config.yaml | Removed pre-push stage specification for tests |
| .github/copilot-instructions.md | Removed comment about not running pre-commit locally |
| /* Custom scrollbar for WebKit browsers */ | ||
| .scrollbar-hide::-webkit-scrollbar { | ||
| width: 0; | ||
| height: 0; | ||
| display: none; | ||
| } | ||
| width: 8px; | ||
| } | ||
| .scrollbar-hide::-webkit-scrollbar-track { | ||
| background: transparent; | ||
| } | ||
| .scrollbar-hide::-webkit-scrollbar-thumb { | ||
| background-color: rgba(156, 163, 175, 0.5); | ||
| border-radius: 4px; | ||
| } | ||
| .scrollbar-hide::-webkit-scrollbar-thumb:hover { | ||
| background-color: rgba(156, 163, 175, 0.7); | ||
| } | ||
| .dark .scrollbar-hide::-webkit-scrollbar-thumb { | ||
| background-color: rgba(107, 114, 128, 0.5); | ||
| } | ||
| .dark .scrollbar-hide::-webkit-scrollbar-thumb:hover { | ||
| background-color: rgba(107, 114, 128, 0.7); | ||
| } |
Copilot
AI
Dec 14, 2025
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.
The class name scrollbar-hide is misleading as the new implementation makes the scrollbar visible rather than hiding it. Consider renaming to scrollbar-custom or scrollbar-styled to better reflect its purpose.
Reverts #5281