This commit adds validation for course advanced settings. Currently when course
administrators make invalid changes in the Settings/Advanced Settings tab,
they're not notified through a new modal window of the list of invalid settings
changes.
* Extending CourseMetadata
- Previously, we only had update_from_json method in CourseMetadata.py,
and it was only validating one field every POST request.
- Now we have validate_and_update_from_json method that encapsulates the
functionality of update_from_json into a validation call
- To avoid discrepancy of validation standards between modules, validation
uses the from_json method implemented to each field in xblock.
* Different Response in advanced settings ajax requests
- After receiving a POST ajax request, course.py calls
validate_and_update_from_json, and sends a json object of either:
1) valid course metadata model
2) error objects
* Error Messages shown in validation-error-modal
- error objects passed through ajax are shown in a separate modal.
| Name |
Last commit
|
Last update |
|---|---|---|
| .. | ||
| cache_toolbox | Loading commit data... | |
| config_models | Loading commit data... | |
| contentserver | Loading commit data... | |
| course_action_state | Loading commit data... | |
| course_groups | Loading commit data... | |
| course_modes | Loading commit data... | |
| dark_lang | Loading commit data... | |
| datadog | Loading commit data... | |
| django_comment_common | Loading commit data... | |
| django_future | Loading commit data... | |
| edxmako | Loading commit data... | |
| embargo | Loading commit data... | |
| external_auth | Loading commit data... | |
| geoinfo | Loading commit data... | |
| heartbeat | Loading commit data... | |
| lang_pref | Loading commit data... | |
| microsite_configuration | Loading commit data... | |
| monitoring | Loading commit data... | |
| pipeline_js | Loading commit data... | |
| pipeline_mako | Loading commit data... | |
| request_cache | Loading commit data... | |
| reverification | Loading commit data... | |
| service_status | Loading commit data... | |
| session_inactivity_timeout | Loading commit data... | |
| static_replace | Loading commit data... | |
| status | Loading commit data... | |
| student | Loading commit data... | |
| terrain | Loading commit data... | |
| third_party_auth | Loading commit data... | |
| track | Loading commit data... | |
| user_api | Loading commit data... | |
| util | Loading commit data... | |
| xmodule_django | Loading commit data... | |
| xmodule_modifiers.py | Loading commit data... |