Merge pull request #9 from Stanford-Online/giulio/preempt-empty-dict-bug

Default value should be empty dict literal string
Stanford-Online · Feb 28, 2017 · fa93bca · fa93bca
2 parents 7790363 + 05de477
commit fa93bca
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 7 deletions.
diff --git a/invideoquiz/invideoquiz.py b/invideoquiz/invideoquiz.py
@@ -48,7 +48,7 @@ class InVideoQuizXBlock(StudioEditableXBlockMixin, XBlock):
 
     timemap = String(
         display_name=_('Problem Timemap'),
-        default={},
+        default='{}',
         scope=Scope.settings,
         help=_(
             'A simple string field to define problem IDs '

diff --git a/invideoquiz/public/html/invideoquiz.html b/invideoquiz/public/html/invideoquiz.html
@@ -1,3 +1 @@
-<div class="in-video-quiz-block" data-videoid="{video_id}" data-mode='{user_mode}'>
-    <p>This is an XBlock to display problem components inside of videos.</p>
-</div>
+<div class="in-video-quiz-block" data-videoid="{video_id}" data-mode='{user_mode}'></div>
diff --git a/invideoquiz/public/js/src/invideoquiz.js b/invideoquiz/public/js/src/invideoquiz.js
@@ -1,5 +1,6 @@
 /* Javascript for InVideoQuizXBlock. */
 function InVideoQuizXBlock(runtime, element) {
+    $('.in-video-quiz-block').closest('.vert').hide();
     var videoId = $('.in-video-quiz-block').data('videoid');
     if (!videoId || !InVideoQuizXBlock.config.hasOwnProperty(videoId)) {
         return;
@@ -19,8 +20,6 @@ function InVideoQuizXBlock(runtime, element) {
     var intervalTimeout = 1500;
 
     $(function () {
-        $('.in-video-quiz-block').closest('.vert').hide();
-
         $('#seq_content .vert-mod .vert').each(function () {
             var component = $(this);
 

diff --git a/setup.py b/setup.py
@@ -45,7 +45,7 @@ def run_tests(self):
 
 setup(
     name='invideoquiz-xblock',
-    version='0.2.0',
+    version='0.1.4',
     description='Helper XBlock to locate CAPA problems within videos.',
     license='AGPL v3',
     packages=[