Add API to support courseware MFE #22848
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
davestgermain
force-pushed
the
dcs/course-api
branch
8 times, most recently
from
cc4475f to
eb63271
Compare
davestgermain
force-pushed
the
dcs/course-api
branch
4 times, most recently
from
5299217 to
fce11e3
Compare
davestgermain
commented
Jan 28, 2020
Contributor
Author
There was a problem hiding this comment.
The intention is to get all of the required information from the CourseOverview, without having to hit the modulestore. But some of the machinery for instantiating tabs checks other attributes on the course. We could decide whether to add these to the overview, or proxy to the course object.
davestgermain
changed the title
davestgermain
force-pushed
the
dcs/course-api
branch
from
fce11e3 to
40a22ba
Compare
davestgermain
force-pushed
the
dcs/course-api
branch
from
40a22ba to
00753ec
Compare
Contributor
There was a problem hiding this comment.
What values can enrollment take?
davidjoy
approved these changes
Jan 28, 2020
davestgermain
force-pushed
the
dcs/course-api
branch
from
00753ec to
1aa8c9c
Compare
|
Your PR has finished running tests. There were no failures. |
Contributor
|
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a new courseware API, at
/api/courseware/course/{course_id}It looks substantially the same as the course API, but does not guarantee any backwards compatibility.
This also adds an API for returning sequence metadata, at
/api/courseware/sequence/{usage_key}which is the same information from #22707TNL-7054