edx-platform
edx-platform copied to clipboard
fix: Course blocks API with param return_type=list 34379
Issue Description
Version: Quince.1 Discussion: edx (New). Endpoint: {LMS_HOST}/api/courses/v1/blocks/?course_id={COURSE_ID}&return_type=list
Console log:
Thanks for the pull request, @qasimgulzar! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:
- supporting documentation
- Open edX discussion forum threads
- timeline information ("this must be merged by XX date", and why that is)
- partner information ("this is a course on edx.org")
- any other information that can help Product understand the context for the PR
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.
Tests have passed, removing the flag.
@mphilbrick211 Looks like the Aurora team owns this Course API, so i'll be pinging them to be aware of this PR!
@mphilbrick211 Looks like the Aurora team owns this Course API, so i'll be pinging them to be aware of this PR!
Thanks, @jsnwesson, though I don't think Aurora is reviewing these anymore.
Approving/merging since this was already approved/merged for the master branch in #34424 , apologies for leaving this hanging for so long!
@qasimgulzar 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.