Security enhacements 785 & 788 - #3439
Closed
ziafazal wants to merge 51 commits into
Closed
Conversation
update hotspot reveal styling address some PR feedback. Start to build out some Jasmine tests address PR comments add some default content fix when running under https
…m type - Added hinting/feedback mechanism for multiple choice problem type - fix styles to make feedback act as an overlay as opposed to a side-by-side block
…euse, forced password resets, allowed frequency of password resets
…urseware. Also, make the GA account ID configurable rather than hard coded in the Mako template Conflicts: lms/envs/aws.py lms/envs/common.py
This updates the XBlock dependency to edx/XBlock@cba8d78 and add the newly introduced enforce_type methods
Included commits: * xblock-external-ui: Include CSRF token in the API answer * xblock-external-ui: Adds support for CORS headers (cross-domain request) * xblock-external-ui: Include full path when building local_url
Xblock external UI - Server-side integration
move api from common to lms
…hash2 drag-n-drop: update hash
Course-Group Relationships
Cdodge/sprint h api work
add ability to enroll users into courses
Cdodge/add group profile
…f courses associated with the group (program)
add a GET endpoint to /api/groups/{groupid}/courses to get the list of c...
return the display names when listing all courses in a group (program)
create optimized course tree query API endpoint
Contributor
|
Wow, this is a massive pull request. Can you break it out into smaller ones? Also, it needs a rebase, the tests are failing, and some docs would be super helpful. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
We added Security enhacements #785 & #788 to api manager.