Skip to content

Remove all OSSH code — migrated to BLT-OSSH static site#6033

Closed
stealthwhizz wants to merge 1 commit intoOWASP-BLT:mainfrom
stealthwhizz:remove-ossh-code
Closed

Remove all OSSH code — migrated to BLT-OSSH static site#6033
stealthwhizz wants to merge 1 commit intoOWASP-BLT:mainfrom
stealthwhizz:remove-ossh-code

Conversation

@stealthwhizz
Copy link
Contributor

@stealthwhizz stealthwhizz commented Mar 19, 2026

Summary

  • OSSH has been fully migrated to the standalone BLT-OSSH static site
  • This PR removes all remaining OSSH backend code from the main BLT repo (~2100 lines deleted)
  • Navigation links (header, sidenav, features page) now point to https://ossh.owaspblt.org

Changes

Deleted files

  • website/views/ossh.py — all OSSH views and recommendation logic
  • website/views/ossh_catalog.py — catalog API endpoint
  • website/templates/ossh/ — all 7 templates (home, results, 5 includes)
  • website/management/commands/fetch_devto_articles.py
  • website/management/commands/fetch_discord_servers.py
  • website/management/commands/fetch_reddit_communities.py
  • website/tests/test_ossh_recommender.py

Modified files

  • blt/urls.py — removed OSSH imports and 8 URL patterns
  • website/models.py — removed OsshCommunity, OsshDiscussionChannel, OsshArticle models
  • website/admin.py — removed OSSH imports, admin classes, and registrations
  • website/tests/test_model_str_methods.py — removed OsshCommunity test
  • website/templates/includes/header.html — link to external OSSH site
  • website/templates/includes/sidenav.html — link to external OSSH site
  • website/templates/features.html — link to external OSSH site and repo

New files

  • website/migrations/0274_remove_ossh_models.py — drops the 3 OSSH tables

Test plan

  • Verify the site loads without errors (no broken imports)
  • Verify header/sidenav OSSH links open https://ossh.owaspblt.org in new tab
  • Verify migration runs cleanly (python manage.py migrate)
  • Run existing tests to confirm no regressions

Closes OWASP-BLT/BLT-OSSH#30

Summary by CodeRabbit

Release Notes

  • Removed Features

    • Removed the internal Open Source Sorting Hat recommendation tool. This feature has been transitioned to a standalone service available at https://ossh.owaspblt.org.
  • Updates

    • Navigation links have been updated to direct users to the external Open Source Sorting Hat service.

OSSH has been migrated to the standalone BLT-OSSH static site
(https://ossh.owaspblt.org). This removes all backend code that
is no longer needed:

- Delete views: ossh.py, ossh_catalog.py
- Delete templates: ossh/ directory (home, results, includes)
- Delete management commands: fetch_devto_articles, fetch_discord_servers, fetch_reddit_communities
- Delete tests: test_ossh_recommender.py
- Remove models: OsshCommunity, OsshDiscussionChannel, OsshArticle
- Remove admin registrations and URL patterns
- Remove OsshCommunity test from test_model_str_methods.py
- Add migration 0274 to drop OSSH tables
- Update header/sidenav/features to link to external OSSH site

Closes OWASP-BLT/BLT-OSSH#30
@owasp-blt
Copy link

owasp-blt bot commented Mar 19, 2026

📊 Monthly Leaderboard

Hi @stealthwhizz! Here's how you rank for March 2026:

Rank User Open PRs PRs (merged) PRs (closed) Reviews Comments Total
7 Kunal241207 @Kunal241207 7 26 0 4 9 305
8 stealthwhizz @stealthwhizz 7 26 0 2 13 303
9 S3DFX-CYBER @S3DFX-CYBER 6 15 12 25 21 299

Scoring this month (across OWASP-BLT org): Open PRs (+1 each), Merged PRs (+10), Closed (not merged) (−2), Reviews (+5; first two per PR in-month), Comments (+2, excludes CodeRabbit). Run /leaderboard on any issue or PR to see your rank!

@owasp-blt owasp-blt bot added unresolved-conversations: 0 PR has 0 unresolved conversations 16 checks pending labels Mar 19, 2026
@github-actions
Copy link
Contributor

👋 Hi @stealthwhizz!

This pull request needs a peer review before it can be merged. Please request a review from a team member who is not:

  • The PR author
  • DonnieBLT
  • coderabbitai
  • copilot

Once a valid peer review is submitted, this check will pass automatically. Thank you!

⚠️ The Check Peer Review workflow check is currently failing until a valid peer review is received.

@github-actions github-actions bot added files-changed: 21 PR changes 21 files migrations PR contains database migration files labels Mar 19, 2026
@owasp-blt owasp-blt bot added the needs-peer-review PR needs peer review label Mar 19, 2026
@github-actions
Copy link
Contributor

👋 Hi @stealthwhizz,

🚫 This pull request has been automatically closed because only first-time contributors (FIRST_TIMER or FIRST_TIME_CONTRIBUTOR) may submit pull requests to this repository at this time.

Your GitHub account has previously contributed to this repository, so this PR does not qualify for automatic acceptance.

⚠️ Please note: Closed pull requests of this type are recorded and will count toward negative scoring on your contributor profile.

If you believe this was closed in error, please open an issue to discuss it with the maintainers.

Thank you for your understanding! 🙏

@github-actions github-actions bot added the linked-issue PR has a linked GitHub issue label Mar 19, 2026
@github-actions github-actions bot closed this Mar 19, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 19, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: OWASP-BLT/coderabbit/.coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 88244f9e-8f11-408d-9ea9-281b03c13a18

📥 Commits

Reviewing files that changed from the base of the PR and between 8b4447c and 30de2e8.

📒 Files selected for processing (21)
  • blt/urls.py
  • website/admin.py
  • website/management/commands/fetch_devto_articles.py
  • website/management/commands/fetch_discord_servers.py
  • website/management/commands/fetch_reddit_communities.py
  • website/migrations/0274_remove_ossh_models.py
  • website/models.py
  • website/templates/features.html
  • website/templates/includes/header.html
  • website/templates/includes/sidenav.html
  • website/templates/ossh/home.html
  • website/templates/ossh/includes/github_stats.html
  • website/templates/ossh/includes/recommended_articles.html
  • website/templates/ossh/includes/recommended_communities.html
  • website/templates/ossh/includes/recommended_discussion_channels.html
  • website/templates/ossh/includes/recommended_repos.html
  • website/templates/ossh/results.html
  • website/tests/test_model_str_methods.py
  • website/tests/test_ossh_recommender.py
  • website/views/ossh.py
  • website/views/ossh_catalog.py

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting


Walkthrough

This PR comprehensively removes all OSSH (Open Source Sorting Hat) code from the main BLT Django repository, including models, views, management commands, templates, admin registrations, URL routes, and tests. Navigation links are updated to point to an external OSSH deployment at https://ossh.owaspblt.org. A database migration drops the three OSSH model tables.

Changes

Cohort / File(s) Summary
Models & Database
website/models.py, website/migrations/0274_remove_ossh_models.py
Removed OsshCommunity, OsshDiscussionChannel, and OsshArticle model classes; added migration to drop all three tables from the database.
Views & Recommendation Logic
website/views/ossh.py, website/views/ossh_catalog.py
Deleted entire OSSH views module including home/results pages, recommendation algorithms, rate limiting, and GitHub data fetching; removed catalog JSON endpoint.
Management Commands
website/management/commands/fetch_devto_articles.py, website/management/commands/fetch_discord_servers.py, website/management/commands/fetch_reddit_communities.py
Removed all three data-fetching management commands for DEV.to articles, Discord servers, and Reddit communities.
Admin Configuration
website/admin.py
Removed model imports, admin classes (OsshArticleAdmin, OsshDiscussionChannelAdmin), and admin registrations for OSSH models.
URL Configuration
blt/urls.py
Removed all OSSH URL route patterns and associated view function imports (ossh_home, ossh_results, get_github_data, get_recommended_* endpoints).
Templates & Navigation
website/templates/ossh/home.html, website/templates/ossh/results.html, website/templates/ossh/includes/*, website/templates/includes/header.html, website/templates/includes/sidenav.html, website/templates/features.html
Deleted all OSSH template pages and includes; updated header, sidenav, and features template navigation links to external URL https://ossh.owaspblt.org with target="_blank" and security attributes.
Tests
website/tests/test_ossh_recommender.py, website/tests/test_model_str_methods.py
Removed entire OSSH test suite and deleted related test case references in model string method tests.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~35 minutes

Suggested labels

quality: high

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

❤️ Share

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 checks pending files-changed: 21 PR changes 21 files linked-issue PR has a linked GitHub issue migrations PR contains database migration files needs-peer-review PR needs peer review quality: high unresolved-conversations: 0 PR has 0 unresolved conversations

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Remove all OSSH code from main BLT Django repo

1 participant