Skip to content

Addition of not responded posts filter in fetch threads lists API. - #31275

Merged
mehaknasir merged 2 commits into
masterfrom
inf-524
Nov 11, 2022
Merged

Addition of not responded posts filter in fetch threads lists API. #31275
mehaknasir merged 2 commits into
masterfrom
inf-524

Conversation

@mehaknasir

Copy link
Copy Markdown
Contributor

Addition of not responded posts filter in fetch threads lists API.

Description

Addition of not responded posts filter in fetch threads lists API.
Jira ticket: https://2u-internal.atlassian.net/browse/INF-524
related FE PR: openedx/frontend-app-discussions#286

Useful information to include:

  • Which edX user roles will this change impact? Common user roles are "Learner", "Course Author".

Screen Shot 2022-11-10 at 2 26 07 AM

Supporting information

Jira ticket: https://2u-internal.atlassian.net/browse/INF-524
related FE PR: openedx/frontend-app-discussions#286

Testing instructions

  • checkout to this branch and make sure you have the latest changes from cs_comment_service as well
  • use mehak/INF-475 branch of discussions MFE
  • run devstack and open the filters section in all posts/ and my posts
  • select not responded from the filters section now scroll through the list and. it should display a list of all such discussion posts which have not been responded to.
  • Click load more posts to validate no such posts are returned in paginated response as well.

@mehaknasir
mehaknasir merged commit b6ee283 into master Nov 11, 2022
@mehaknasir
mehaknasir deleted the inf-524 branch November 11, 2022 09:56
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

3 participants