Skip to content

Commit b4897ed

Browse files
Taimoor  Ahmedormsbee
authored andcommitted
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.
1 parent 009dd30 commit b4897ed

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

  • openedx/core/djangoapps/django_comment_common/comment_client

openedx/core/djangoapps/django_comment_common/comment_client/user.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -169,8 +169,6 @@ def subscribed_threads(self, query_params=None):
169169
params.update(query_params)
170170
course_key = utils.get_course_key(self.attributes.get("course_id"))
171171
if is_forum_v2_enabled(course_key):
172-
if user_id := params.get("user_id"):
173-
params["user_id"] = str(user_id)
174172
if page := params.get("page"):
175173
params["page"] = int(page)
176174
if per_page := params.get("per_page"):
@@ -179,7 +177,11 @@ def subscribed_threads(self, query_params=None):
179177
params["count_flagged"] = str_to_bool(count_flagged)
180178
if not params.get("course_id"):
181179
params["course_id"] = str(course_key)
182-
response = forum_api.get_user_threads(**params)
180+
181+
user_id = params.pop("user_id", None)
182+
if "text" in params:
183+
params.pop("text")
184+
response = forum_api.get_user_subscriptions(user_id, str(course_key), params)
183185
else:
184186
response = utils.perform_request(
185187
'get',

0 commit comments

Comments
 (0)