Skip to content

fix: Optimize MySQL query for getting voted_ids list#53

Merged
Alam-2U merged 1 commit into
edx:release-ulmofrom
rahulkanneri-2u:cosmo2-849-optimize-mysql-query-for-voted-ids
May 22, 2026
Merged

fix: Optimize MySQL query for getting voted_ids list#53
Alam-2U merged 1 commit into
edx:release-ulmofrom
rahulkanneri-2u:cosmo2-849-optimize-mysql-query-for-voted-ids

Conversation

@rahulkanneri-2u
Copy link
Copy Markdown

@rahulkanneri-2u rahulkanneri-2u commented May 20, 2026

Description:

Updated the MySQL backend to return course-specific upvoted and downvoted IDs in the complete user response.
This prevents vote IDs from other courses from being included when a course_id is provided.

Changes Included:

  • Added optional course_id filtering to get_user_voted_ids().
  • Filtered votes by related CommentThread and Comment records for the given course.
  • Passed course_id when fetching upvoted_ids and downvoted_ids in the complete user response.
  • Bumped forum package version from 0.6.3 to 0.6.4.

Tickets:

COSMO2-849

@rahulkanneri-2u rahulkanneri-2u force-pushed the cosmo2-849-optimize-mysql-query-for-voted-ids branch from 49566e3 to 61afb43 Compare May 21, 2026 05:47
@rahulkanneri-2u rahulkanneri-2u force-pushed the cosmo2-849-optimize-mysql-query-for-voted-ids branch from 61afb43 to 3fd5867 Compare May 21, 2026 09:37
@Alam-2U Alam-2U merged commit 8ee9db5 into edx:release-ulmo May 22, 2026
13 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.

2 participants