Move entity channel admin to owasp app and fix its bug

[ahmedxgouda]

Proposed change

Checklist

• I've read and followed the contributing guidelines.
• I've run make check-test locally; all checks and tests passed.

[coderabbitai]

Summary by CodeRabbit

• Refactor
  • Streamlined the OWASP admin interface by removing redundant search controls from the Entity Channel admin, resulting in a cleaner and simpler form.
• Chores
  • Consolidated admin registrations under the OWASP admin package and removed a redundant export from the Slack admin to reduce duplication and improve maintainability.

Walkthrough

Moves EntityChannelAdmin admin wiring from slack.admin to owasp.admin, adds its import in OWASP admin init, removes its export from Slack admin init, and updates EntityChannelAdmin by removing the search_button and channel_search fields.

Changes

Cohort / File(s)	Summary
OWASP admin wiring backend/apps/owasp/admin/__init__.py	Adds import for EntityChannelAdmin from .entity_channel.
EntityChannelAdmin field adjustments backend/apps/owasp/admin/entity_channel.py	Removes search_button and channel_search from EntityChannelAdmin fields.
Slack admin export removal backend/apps/slack/admin/__init__.py	Removes export/import of EntityChannelAdmin.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Fix #2019: Link OWASP entities to related Slack channels #2049 — Previously introduced EntityChannelAdmin under slack.admin with fields now being removed and is likely the origin of the wiring being relocated to owasp.admin.

Suggested reviewers

• arkid15r
• kasya

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description Check	❓ Inconclusive	The description currently contains only the default template and lacks any details about the specific changes, making it too generic to convey the intent and scope of the PR.	Please update the description to include a summary of the actual changes made, such as moving the EntityChannelAdmin to the OWASP app and removing obsolete fields to address the related bug.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title concisely states the main purpose of the PR by indicating that the entity channel admin is being moved to the OWASP app and that a related bug is being fixed, which directly reflects the core changes made in the diff.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0e37466 and ac1fd41.

📒 Files selected for processing (3)

• backend/apps/owasp/admin/__init__.py (1 hunks)
• backend/apps/owasp/admin/entity_channel.py (0 hunks)
• backend/apps/slack/admin/__init__.py (0 hunks)

💤 Files with no reviewable changes (2)

• backend/apps/owasp/admin/entity_channel.py
• backend/apps/slack/admin/init.py

🧰 Additional context used

🧬 Code graph analysis (1)

backend/apps/owasp/admin/__init__.py (1)

backend/apps/owasp/admin/entity_channel.py (1)

• EntityChannelAdmin (20-82)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: Run frontend unit tests
• GitHub Check: Run backend tests
• GitHub Check: Run frontend e2e tests

🔇 Additional comments (1)

backend/apps/owasp/admin/__init__.py (1)

9-9: Approve EntityChannelAdmin import and registration.
EntityChannelAdmin is registered via @admin.register(EntityChannel) in entity_channel.py, the model lives in backend/apps/owasp/models, and no stale references remain in slack/admin.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud