Skip to content

Planning file deletion strategy#6044

Open
Copilot wants to merge 1 commit intomainfrom
copilot/delete-unnecessary-files
Open

Planning file deletion strategy#6044
Copilot wants to merge 1 commit intomainfrom
copilot/delete-unnecessary-files

Conversation

Copy link
Contributor

Copilot AI commented Mar 21, 2026

Pull request created by AI Agent

…e test_feed Co-authored-by: DonnieBLT <128622481+DonnieBLT@users.noreply.github.com> Agent-Logs-Url: https://github.com/OWASP-BLT/BLT/sessions/61b29a84-29dc-4fee-b8a8-e13c7608579c
@github-actions github-actions bot added unresolved-conversations: 0 PR has 0 unresolved conversations files-changed: 6 PR changes 6 files labels Mar 21, 2026
@DonnieBLT DonnieBLT marked this pull request as ready for review March 21, 2026 00:48
Copilot AI review requested due to automatic review settings March 21, 2026 00:49
@github-actions github-actions bot added the security: review needed Security scans need review label Mar 21, 2026
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 PR removes a few placeholder/template-only endpoints and simplifies routing by replacing trivial view functions with TemplateView, while adding automated coverage for the global activity feed and its admin-only delete behavior.

Changes:

  • Removed the dedicated video_call view module and routed /video_call/ directly to the template via TemplateView.
  • Switched /teams/ and /apps/ to render coming_soon.html and removed the now-unneeded placeholder templates.
  • Added a new test module covering the feed page, RSS feed, and superuser-only activity deletion.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
website/views/video_call.py Deleted trivial render-only view.
blt/urls.py Updates routes for video_call, teams, and apps to use TemplateView / coming_soon.html.
website/tests/test_feed.py Adds test coverage for feed rendering, RSS feed, and delete authorization.
website/templates/teams.html Removed unused placeholder template.
website/templates/apps.html Removed unused placeholder template.
website/templates/deletions.html Removed unused placeholder template.
@github-actions github-actions bot added pre-commit: passed Pre-commit checks passed tests: P:657 F:0 S:2 Django tests passed 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 Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

files-changed: 6 PR changes 6 files last-active: 1d PR last updated 1 day ago pre-commit: passed Pre-commit checks passed security: review needed Security scans need review tests: P:657 F:0 S:2 Django tests passed unresolved-conversations: 0 PR has 0 unresolved conversations

3 participants