Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 2 commits into from
Nov 11, 2022

Conversation

mehaknasir
Copy link
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
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
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