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

Skip to content

[WIP] Update changes based on PR 5284 #5298

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.

Sign up for GitHub

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

Jump to bottom
Draft
Copilot wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

[WIP] Update changes based on PR 5284 #5298

Copilot wants to merge 3 commits into from

Conversation

Copy link
Contributor

Copilot AI commented Dec 15, 2025 •
edited
Loading

✅ Successfully Applied All Changes from PR #5284

  • Fetched PR Updates dec14 #5284 branch (revert-5283-revert-5281-updates_dec14)
  • Applied all 27 file changes to current branch
  • Verified all new files are present
  • Confirmed file deletion (project_channels.csv)
  • All changes committed and ready

📊 Changes Applied

Files Changed: 27 files

  • Modified: 23 files
  • Added: 3 new files (populate_github_org.py, organization_list.js, organization_list_mode.html)
  • Deleted: 1 file (project_channels.csv)

Statistics:

  • Lines added: ~2,579
  • Lines deleted: ~1,186
  • Net change: +1,393 lines

✨ Features Included

🎉 New Features

  1. Organization Repo Refresh API - /api/organization/<org_id>/refresh/ endpoint with live UI updates
  2. Organization List Mode - Table view at /organizations/list/ with 22 sortable columns
  3. GitHub Org Population Command - python manage.py populate_github_org CLI tool
  4. Enhanced Repo Data Refresh - Issues, PRs, releases, and topics sync from GitHub

🎨 UI/UX Improvements

  • Sidebar pin toggle with localStorage persistence
  • Conditional organization navigation links
  • Breadcrumbs on repo list page
  • GSOC badge styling with Google colors
  • Hidden scrollbars for cleaner UI
  • Enhanced management commands layout
  • Better dark mode support

🐛 Bug Fixes

  • Throttling bypassed in DEBUG mode
  • Console email backend for development
  • Proper GitHub hostname validation (security fix)
  • Better error message sanitization
  • Added rel="noopener noreferrer" to external links

🔧 Configuration Changes

  • Pre-commit tests moved to pre-push stage
  • Startup script runs migrations automatically
  • Browser auto-opens on startup
  • Removed unused project_channels.csv

✅ Validation

All changes have been successfully applied:

  • ✅ 3 new files created and present
  • ✅ 23 files modified with PR Updates dec14 #5284 changes
  • ✅ 1 file deleted (project_channels.csv)
  • ✅ Python syntax validated
  • ✅ Git history clean
  • ✅ Ready for review and testing

This PR now contains all changes from the original PR #5284 and is ready for CI/CD checks and final review.

Original prompt

Create a new pr based in this one #5284

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 15, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Note

Free review on us!

CodeRabbit is offering free reviews until Wed Dec 17 2025 to showcase some of the refinements we've made.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI requested a review from DonnieBLT December 15, 2025 10:13
@DonnieBLT
Copy link
Collaborator

DonnieBLT commented Dec 15, 2025

@copilot Try agsin

@DonnieBLT
Apply all changes from PR #5284
405b7f5 
This commit includes all 27 files changed from PR #5284:
- Organization repo refresh API/UI
- Organization list mode table view
- GitHub org population command
- Enhanced repo data refresh
- Sidebar improvements with pin toggle
- Conditional navigation links
- Management commands enhancements
- Bug fixes and configuration changes

Co-authored-by: DonnieBLT <[email protected]>
Copy link
Contributor Author

Copilot AI commented Dec 15, 2025

@copilot Try agsin

Done! Successfully applied all 27 files from PR #5284 to this branch. Changes include:

  • Organization repo refresh API/UI
  • Organization list mode table view
  • GitHub org population command
  • Enhanced repo data refresh
  • Sidebar pin toggle with localStorage
  • Conditional navigation links
  • Management commands improvements
  • Bug fixes and configuration changes

Commit: 405b7f5

@github-actions github-actions bot added the files-changed: 27 PR changes 27 files label Dec 15, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025

❌ Pre-commit checks failed

The pre-commit hooks found issues that need to be fixed. Please run the following commands locally to fix them:

# Install pre-commit if you haven't already
pip install pre-commit

# Run pre-commit on all files
pre-commit run --all-files

# Or run pre-commit on staged files only
pre-commit run

After running these commands, the pre-commit hooks will automatically fix most issues.
Please review the changes, commit them, and push to your branch.

💡 Tip: You can set up pre-commit to run automatically on every commit by running:

pre-commit install
Pre-commit output 
[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
[WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version.  Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this.  if it does not -- consider reporting an issue to that repo.
[INFO] Initializing environment for https://github.com/pycqa/isort.
[WARNING] repo `https://github.com/pycqa/isort` uses deprecated stage names (commit, merge-commit, push) which will be removed in a future version.  Hint: often `pre-commit autoupdate --repo https://github.com/pycqa/isort` will fix this.  if it does not -- consider reporting an issue to that repo.
[INFO] Initializing environment for https://github.com/astral-sh/ruff-pre-commit.
[INFO] Initializing environment for https://github.com/djlint/djLint.
[INFO] Initializing environment for local.
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/pycqa/isort.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/astral-sh/ruff-pre-commit.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/djlint/djLint.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for local.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
check python ast.........................................................Passed
check builtin type constructor use.......................................Passed
check yaml...............................................................Passed
fix python encoding pragma...............................................Passed
mixed line ending........................................................Passed
isort....................................................................Passed
ruff.....................................................................Failed
- hook id: ruff
- files were modified by this hook

Found 3 errors (3 fixed, 0 remaining).

For more information, see the pre-commit documentation.

@github-actions github-actions bot added pre-commit: failed Pre-commit checks failed tests: passed Django tests passed labels Dec 15, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025

❌ Pre-commit checks failed

The pre-commit hooks found issues that need to be fixed. Please run the following commands locally to fix them:

# Install pre-commit if you haven't already
pip install pre-commit

# Run pre-commit on all files
pre-commit run --all-files

# Or run pre-commit on staged files only
pre-commit run

After running these commands, the pre-commit hooks will automatically fix most issues.
Please review the changes, commit them, and push to your branch.

💡 Tip: You can set up pre-commit to run automatically on every commit by running:

pre-commit install
Pre-commit output 
[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
[WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version.  Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this.  if it does not -- consider reporting an issue to that repo.
[INFO] Initializing environment for https://github.com/pycqa/isort.
[WARNING] repo `https://github.com/pycqa/isort` uses deprecated stage names (commit, merge-commit, push) which will be removed in a future version.  Hint: often `pre-commit autoupdate --repo https://github.com/pycqa/isort` will fix this.  if it does not -- consider reporting an issue to that repo.
[INFO] Initializing environment for https://github.com/astral-sh/ruff-pre-commit.
[INFO] Initializing environment for https://github.com/djlint/djLint.
[INFO] Initializing environment for local.
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/pycqa/isort.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/astral-sh/ruff-pre-commit.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/djlint/djLint.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for local.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
check python ast.........................................................Passed
check builtin type constructor use.......................................Passed
check yaml...............................................................Passed
fix python encoding pragma...............................................Passed
mixed line ending........................................................Passed
isort....................................................................Passed
ruff.....................................................................Failed
- hook id: ruff
- files were modified by this hook

Found 3 errors (3 fixed, 0 remaining).

For more information, see the pre-commit documentation.

@github-actions github-actions bot added last-active: 0d PR last updated 0 days ago last-active: 1d PR last updated 1 day ago and removed last-active: 0d PR last updated 0 days ago labels Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DonnieBLT DonnieBLT Awaiting requested review from DonnieBLT

Assignees

@DonnieBLT DonnieBLT

Copilot code review Copilot

Labels

files-changed: 27 PR changes 27 files last-active: 1d PR last updated 1 day ago pre-commit: failed Pre-commit checks failed tests: passed Django tests passed

Projects

📌 OWASP BLT Project Board
Status: Backlog

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DonnieBLT