Backport the upstreamed Bulk Enroll API - #64
Conversation
9cbfb4c to
e3e8dbe
Compare
|
Note: I removed this fix, which is not compatible with the version of DRF used in Ficus: https://github.com/edx/edx-platform/pull/15584 See this PR for details!: https://github.com/edx/edx-platform/pull/15631 |
|
@bdero Does it make sense to include both https://github.com/edx/edx-platform/pull/15584 and https://github.com/edx/edx-platform/pull/15631 in this change? Since setting the |
|
@pomegranited There was some pretty significant refactoring that happened between the version of DRF that Cloudera is using and edx:edx-platform/master. Since the DRF request POST parameters are not a property in the Cloudera version of DRF, I suspect that setting the content type in the underlying Django request won't result in the POST params being pulled from the data dict. (It's possible this behavior still exists through a dunder hook or similar, but I couldn't find anything relevant in the code) |
|
@bdero No worries, thanks for your explanation.
|
|
@pomegranited Thank you! |
Part of: https://tasks.opencraft.com/browse/OC-2453
Cherry-picked from PRs:
https://github.com/edx/edx-platform/pull/15584