Skip to content

fix: mending discussion API topic filtering in Teak release (#318)#319

Merged
DeimerM merged 1 commit into
open-release/teak.ceibalfrom
dmh/fix-discussions-issue-teak
May 25, 2026
Merged

fix: mending discussion API topic filtering in Teak release (#318)#319
DeimerM merged 1 commit into
open-release/teak.ceibalfrom
dmh/fix-discussions-issue-teak

Conversation

@DeimerM
Copy link
Copy Markdown

@DeimerM DeimerM commented May 25, 2026

Backport: c1b0228

* fix: stoped group TA to see the posts of other cohorts (openedx#36765)

* fix: stoped group TA to see the posts of other cohorts

* test: updated test cases

---------

Co-authored-by: Ayesha Waris <ayesha.waris@192.168.10.31>

* feat!: remove cs_comments_service support for forums pin API

This will force the use of the new v2 forums API for pinning/unpinning.

* feat!: remove cs_comments_service support for forum's flag APIs

This will force the use of the new v2 forum's APIs for flaging/unflaging.

* feat!: remove cs_comments_service support for forum's vote APIs

This will force the use of the new v2 forum's APIs for voting/unvoting.

* fix: Discussion following posts filter
This PR fixes following filter on the discussions module by replacing  forum api to  api since we only need threads that are subscribed/followed by the requesting user.

* feat!: remove cs_comments_service support for forum's search APIs

This will force the use of the new v2 forum's APIs for searching.

* feat!: remove cs_comments_service support for forum's subscription APIs
- This will force the use of the new v2 forum's APIs for subscriptions.

---------

Co-authored-by: ayesha waris <73840786+ayesha-waris@users.noreply.github.com>
Co-authored-by: Ayesha Waris <ayesha.waris@192.168.10.31>
Co-authored-by: Taimoor Ahmed <68893403+taimoor-ahmed-1@users.noreply.github.com>
Co-authored-by: Ali-Salman29 <ali.salman@arbisoft.com>
Co-authored-by: Taimoor  Ahmed <taimoor.ahmed@A006-01711.local>
@DeimerM DeimerM merged commit 9812d5c into open-release/teak.ceibal May 25, 2026
46 of 47 checks passed
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.

1 participant