move ipywidget check to template

checking the metadata does not work with the execution post-processor
chrisjsewell · Nov 10, 2019 · c468920 · c468920
1 parent e0b7625
commit c468920
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/ipypublish/convert/main.py b/ipypublish/convert/main.py
@@ -426,7 +426,7 @@ def publish(self, ipynb_path, nb_node=None):
         else:
             final_nb, meta_path = (nb_node, ipynb_path)
 
-        # valdate the notebook metadata against the schema
+        # validate the notebook metadata against the schema
         if self.validate_nb_metadata:
             nb_metadata_schema = read_file_from_directory(
                 get_module_path(schema),
@@ -472,12 +472,6 @@ def publish(self, ipynb_path, nb_node=None):
             final_nb, exporter_cls, econfig, jinja_template
         )
 
-        # record if the notebook contains widgets (for use by sphinx)
-        if "application/vnd.jupyter.widget-state+json" in final_nb.metadata.get(
-            "widgets", {}
-        ):
-            resources["contains_ipywidgets"] = True
-
         # postprocess results
         main_filepath = os.path.join(outdir, ipynb_name + exporter.file_extension)
 

diff --git a/ipypublish/templates/segments/ipy-sphinx.yaml.j2 b/ipypublish/templates/segments/ipy-sphinx.yaml.j2
@@ -237,6 +237,7 @@
             <script type="application/vnd.jupyter.widget-state+json">
             {{ nb.metadata.widgets['application/vnd.jupyter.widget-state+json'] | json_dumps }}
             </script>
+        {% set x = resources.__setitem__("contains_ipywidgets", True) %}
         {% endif %}
 
     "jinja_macros": |