Skip to content

Add lti consumer xblock modules to LTI REST endpoints - #15478

Merged
robrap merged 1 commit into
masterfrom
ubc-add-lti-consumer-module-to-rest-endpoints
Jul 6, 2017
Merged

Add lti consumer xblock modules to LTI REST endpoints#15478
robrap merged 1 commit into
masterfrom
ubc-add-lti-consumer-module-to-rest-endpoints

Conversation

@robrap

@robrap robrap commented Jul 6, 2017

Copy link
Copy Markdown
Contributor

This allows LTI consumer xblock modules being hocked into get_course_lti_endpoints for
discovering LTI result/outcome endpoints.

This was originally part of https://openedx.atlassian.net/browse/OSPR-1490

@robrap

robrap commented Jul 6, 2017

Copy link
Copy Markdown
Contributor Author

@dianakhuang This was already approved in this past PR. I will set up a sandbox just in case, but could you provide a thumb on this?

@robrap
robrap requested a review from dianakhuang July 6, 2017 14:26
This allows LTI consumer xblock modules being hocked into get_course_lti_endpoints for
discovering LTI result/outcome endpoints.
@robrap
robrap force-pushed the ubc-add-lti-consumer-module-to-rest-endpoints branch from 4f783bb to 4db8c7a Compare July 6, 2017 14:41
@robrap

robrap commented Jul 6, 2017

Copy link
Copy Markdown
Contributor Author

FYI: I retested this on my sandbox and all looks good. Ready to merge. https://robrap.sandbox.edx.org/courses/course-v1:edX+Test101+course/lti_rest_endpoints/

@robrap
robrap merged commit 0b7ace2 into master Jul 6, 2017
@robrap
robrap deleted the ubc-add-lti-consumer-module-to-rest-endpoints branch July 6, 2017 15:47
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production on Friday, July 07, 2017.

@edx-pipeline-bot

Copy link
Copy Markdown
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.

4 participants