Skip to content

Commit

Permalink
Exclude build_log_url validation in the aplus-json course import
Browse files Browse the repository at this point in the history
Django's URL validator does not accept dotless domain names
such as "grader", which is used in the local testing
run-mooc-grader container.
  • Loading branch information
markkuriekkinen committed Apr 6, 2021
1 parent fbed17c commit 752afbc
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion edit_course/operations/configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -411,7 +411,7 @@ def configure_content(instance, url):
instance.build_log_url = str(config['build_log_url']) if 'build_log_url' in config else ''
# configure_url excluded from validation because the default Django URL
# validation does not accept dotless domain names such as "grader"
instance.full_clean(exclude=['configure_url'])
instance.full_clean(exclude=['configure_url', 'build_log_url'])
instance.save()

if not "categories" in config or not isinstance(config["categories"], dict):
Expand Down

0 comments on commit 752afbc

Please sign in to comment.