Skip to content

feat: bulkdelete ban feature in forum#16

Merged
mraman-2U merged 1 commit intorelease-ulmofrom
mraman/bulkdelete-ban-forum-ulmo
Mar 4, 2026
Merged

feat: bulkdelete ban feature in forum#16
mraman-2U merged 1 commit intorelease-ulmofrom
mraman/bulkdelete-ban-forum-ulmo

Conversation

@mraman-2U
Copy link
Member

This pull request introduces a comprehensive discussion ban management system to the forum API. It adds new API endpoints, serializers, and supporting backend integrations to allow moderators to ban and unban users at both course and organization levels, view ban details, and list banned users. The changes also include the necessary serializers for input validation and output formatting, as well as updates to backend model imports and project dependencies.

Discussion Ban API Implementation:

  • Added new API endpoints for banning users, unbanning users, listing banned users, and retrieving ban details in forum/urls.py and implemented corresponding API views in forum/views/bans.py. These endpoints handle input validation, error handling, and interact with the ban management logic. [1] [2] [3] [4]

  • Introduced serializers for ban-related operations in forum/serializers/bans.py, including BanUserSerializer, UnbanUserSerializer, and response serializers for consistent API input/output.

API and Backend Integration:

  • Exposed new ban management functions (ban_user, unban_user, get_ban, get_banned_users, etc.) in forum/api/__init__.py for use by the API views and other modules. [1] [2] [3] [4]

  • Added imports for MongoDB ban-related models (DiscussionBanExceptions, DiscussionBans, DiscussionModerationLogs) in forum/backends/mongodb/__init__.py to support future backend operations. [1] [2]

Dependencies:

  • Added django-model-utils to requirements/base.in to support model features used in ban management (such as TimeStampedModel).Merge checklist:
    Check off if complete or not applicable:
  • Version bumped
  • Changelog record added
  • Documentation updated (not only docstrings)
  • Fixup commits are squashed away
  • Unit tests added/updated
  • Manual testing instructions provided
  • Noted any: Concerns, dependencies, migration issues, deadlines, tickets

Copy link
Collaborator

@jcapphelix jcapphelix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm,

Before merging, do bump the version of forum, otherwise changes may not get reflected.

@mraman-2U mraman-2U force-pushed the mraman/bulkdelete-ban-forum-ulmo branch from ed4b8f7 to 65b19d5 Compare February 17, 2026 10:33
@mraman-2U mraman-2U force-pushed the mraman/bulkdelete-ban-forum-ulmo branch from 65b19d5 to ed515cc Compare February 17, 2026 10:40
@mraman-2U mraman-2U marked this pull request as ready for review February 17, 2026 10:44
@mraman-2U mraman-2U merged commit 266f5c2 into release-ulmo Mar 4, 2026
13 checks passed
@mraman-2U mraman-2U deleted the mraman/bulkdelete-ban-forum-ulmo branch March 4, 2026 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants