Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@DonnieBLT
Copy link
Collaborator

Reverts #5281

Copilot AI review requested due to automatic review settings December 14, 2025 07:15
@github-actions
Copy link
Contributor

❌ Console Statements Detected

Found 1 file(s) with console statements that should be removed.

Why remove console statements?

  • The project has sufficient error tracking in place
  • Console statements can expose sensitive information in production
  • They can clutter browser console logs
  • They may impact performance in production builds

Files with violations:

website/static/js/repo_detail.js:

29:        console.error('Failed to copy text: ', err);
100:            console.error('CSRF token not found. Make sure cookies are enabled or the CSRF meta tag exists.');
129:            console.error('Error parsing response:', parseError);
312:        console.error('Error:', error);
379:            console.error('Error:', error);
443:        console.error('Error fetching stargazers:', error);

How to fix:

  1. Remove or comment out all console.* statements from your code (console.log, console.error, console.warn, etc.)
  2. Use the project's existing error tracking system for debugging
  3. For temporary debugging during development, comment out console statements: // console.log()

Note:

  • All console methods (log, error, warn, debug, info, etc.) are forbidden.
  • If console statements are inside a multi-line comment block (/* ... */), they may still be flagged. Please manually verify.

Please remove all console statements and push your changes.

@github-actions github-actions bot added the has-console-statements PR contains console statements that need to be removed label Dec 14, 2025
@DonnieBLT DonnieBLT merged commit 8377771 into main Dec 14, 2025
18 of 19 checks passed
@DonnieBLT DonnieBLT deleted the revert-5281-updates_dec14 branch December 14, 2025 07:15
@github-actions github-actions bot added the files-changed: 27 PR changes 27 files label Dec 14, 2025
@github-actions github-actions bot added the pre-commit: passed Pre-commit checks passed label Dec 14, 2025
Copy link
Contributor

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

Comment on lines 8 to 20
/* 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);
}
Copy link

Copilot AI Dec 14, 2025

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.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

files-changed: 27 PR changes 27 files has-console-statements PR contains console statements that need to be removed pre-commit: passed Pre-commit checks passed tests: passed Django tests passed

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants