Merged
Conversation
jcapphelix
force-pushed
the
cjoshi/feat-COSMO2-742
branch
from
959218b to
ac1f216
Compare
jcapphelix
force-pushed
the
cjoshi/feat-COSMO2-742
branch
from
e4e8678 to
de02bda
Compare
Alam-2U
approved these changes
Jan 6, 2026
jcapphelix
added a commit
that referenced
this pull request
Jan 7, 2026
This reverts commit 8f6699d.
Alam-2U
pushed a commit
that referenced
this pull request
Jan 7, 2026
This was referenced Jan 12, 2026
mraman-2U
added a commit
that referenced
this pull request
Feb 10, 2026
feat(discussion): Refactor ban and unban API views, models, and tests test(discussion): remove duplicate MySQL test_discussion_ban_models.py style: fix trailing whitespace and long lines in ban files - Remove trailing whitespace from forum/api/bans.py and forum/views/bans.py - Break long lines to meet 120 character limit - Add mypy ignore_errors for ban-related modules (type annotations needed) style: fix trailing whitespace and long lines in ban files - Remove trailing whitespace from forum/api/bans.py and forum/views/bans.py - Break long lines to meet 120 character limit - Add mypy ignore_errors for ban-related modules (type annotations needed) Revert "style: fix trailing whitespace and long lines in ban files" This reverts commit 9f0b0f5. feat(discussion): Refactor discussion ban tests for consistency and readability feat(ban): Refactor ban API functions and add comprehensive tests feat: soft delete feature (#4) Implements soft delete functionality for discussion threads, responses, and comments using the is_deleted flag instead of permanently deleting records. This enables safe deletion and restoration of discussion content while preserving existing data. Revert "feat: soft delete feature (#4)" (#9) This reverts commit 8f6699d. feat: added soft delete functionality (#10) Implements soft delete functionality for discussion threads, responses, and comments using the is_deleted flag instead of permanently deleting records. This enables safe deletion and restoration of discussion content while preserving existing data. feat(ban): Enhance ban and unban APIs to handle User objects directly refactor(ban): Improve code formatting and readability in ban API feat(ban): Update get_banned_users to exclude org-level bans with course exceptions
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Implements soft delete functionality for discussion threads, responses, and comments using the
is_deletedflag instead of permanently deleting records.This enables safe deletion and restoration of discussion content while preserving existing data.
Changes Made
JIRA Tickets
Related Pull Requests
Merge checklist:
Check off if complete or not applicable: