-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Open
Description
It was originally implemented by:
Issues with the original implementation:
- It makes user-facing changes but did not go through community product review
- Documentation incorrectly states that deleted posts are limited to course_id; they actually span across the entire org containing course_id if course_or_org=="org".
- Related to ^, it allows anyone with staff or forum mod role in
course-v1:MyOrg+X+Yto bulk-delete posts across all ofMyOrg. This is a security flaw. - Does not use api_doc_tools; documentation can only be viewed in the codebase.
- Does not work with MySQL-backed forums
- Adds utility code which reaches directly in MongoDB-backed forums interface
We are reverting the frontend changes:
- revert: feat: added bulk delete user posts feature for privileged users frontend-app-discussions#818
For the backend, we will do one of the following:
- Minimizing impact by down-scoping the backend API to global staff only: temp: Restrict forum bulk-delete to global staff only #37401
- Put it behind a feature flag
- Revert the backend API entirely: revert: feat: added endpoint for priviledged roles to delete threads of a user #37403
- Revert the API in Ulmo but leave it in master
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels