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

feat: list courses details by keys #548

Conversation

yusuf-musleh
Copy link

Backport of openedx#32573

This adds the ability to get a list of detailed courses based on their keys provided in the newly added keys query param in the GET /courses/v1/courses/ endpoint.

(cherry picked from commit b14cf0cc76928d3903eafcfda47dbefb44044a7a)

This adds the ability to get a list of detailed courses based on their
keys provided in the newly added `keys` query param in the `GET /courses/v1/courses/`
endpoint.

(cherry picked from commit b14cf0cc76928d3903eafcfda47dbefb44044a7a)
Copy link
Member

@navinkarkera navinkarkera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yusuf-musleh 👍

  • I tested this: (API in nutmeg devstack with filters)
  • I read through the code
  • I checked for accessibility issues
  • Includes documentation
  • I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository.

@yusuf-musleh yusuf-musleh merged commit b26683f into opencraft-release/nutmeg.2 Jul 5, 2023
30 checks passed
@yusuf-musleh yusuf-musleh deleted the yusuf-musleh/backport-BB-7514-list-courses-details-by-key branch July 5, 2023 12:20
@Agrendalath Agrendalath changed the title feat: list courses details by keys https://github.com/openedx/edx-platform/pull/32573 Jul 7, 2023
@Agrendalath Agrendalath changed the title https://github.com/openedx/edx-platform/pull/32573 feat: list courses details by keys Jul 7, 2023
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