diff --git a/cms/djangoapps/pipeline_js/utils.py b/cms/djangoapps/pipeline_js/utils.py deleted file mode 100644 index 2584b023acdf..000000000000 --- a/cms/djangoapps/pipeline_js/utils.py +++ /dev/null @@ -1,19 +0,0 @@ -""" -Utilities for returning XModule JS (used by requirejs) -""" - - -from django.conf import settings -from django.contrib.staticfiles.storage import staticfiles_storage - - -def get_xmodule_urls(): - """ - Returns a list of the URLs to hit to grab all the XModule JS - """ - pipeline_js_settings = settings.PIPELINE['JAVASCRIPT']["module-js"] - if settings.DEBUG: - paths = [path.replace(".coffee", ".js") for path in pipeline_js_settings["source_filenames"]] - else: - paths = [pipeline_js_settings["output_filename"]] - return [staticfiles_storage.url(path) for path in paths] diff --git a/cms/envs/common.py b/cms/envs/common.py index e3e321c56836..bed1862fabd1 100644 --- a/cms/envs/common.py +++ b/cms/envs/common.py @@ -1383,15 +1383,6 @@ 'source_filenames': base_vendor_js, 'output_filename': 'js/cms-base-vendor.js', }, - 'module-js': { - 'source_filenames': ( - rooted_glob(COMMON_ROOT / 'static/', 'xmodule/descriptors/js/*.js') + - rooted_glob(COMMON_ROOT / 'static/', 'xmodule/modules/js/*.js') + - rooted_glob(COMMON_ROOT / 'static/', 'common/js/discussion/*.js') - ), - 'output_filename': 'js/cms-modules.js', - 'test_order': 1 - }, } STATICFILES_IGNORE_PATTERNS = ( diff --git a/lms/envs/common.py b/lms/envs/common.py index c9b7a9f32eee..f366d09dd453 100644 --- a/lms/envs/common.py +++ b/lms/envs/common.py @@ -2586,14 +2586,6 @@ def _make_locale_paths(settings): # pylint: disable=missing-function-docstring 'source_filenames': main_vendor_js, 'output_filename': 'js/lms-main_vendor.js', }, - 'module-descriptor-js': { - 'source_filenames': rooted_glob(COMMON_ROOT / 'static/', 'xmodule/descriptors/js/*.js'), - 'output_filename': 'js/lms-module-descriptors.js', - }, - 'module-js': { - 'source_filenames': rooted_glob(COMMON_ROOT / 'static', 'xmodule/modules/js/*.js'), - 'output_filename': 'js/lms-modules.js', - }, 'discussion': { 'source_filenames': discussion_js, 'output_filename': 'js/discussion.js', diff --git a/lms/templates/instructor/instructor_dashboard_2/instructor_dashboard_2.html b/lms/templates/instructor/instructor_dashboard_2/instructor_dashboard_2.html index 7f0046a79daf..c0392f32933a 100644 --- a/lms/templates/instructor/instructor_dashboard_2/instructor_dashboard_2.html +++ b/lms/templates/instructor/instructor_dashboard_2/instructor_dashboard_2.html @@ -54,7 +54,8 @@ - <%static:js group='module-descriptor-js'/> + <%static:webpack entry='HtmlBlockEditor'/> + <%static:js group='instructor_dash'/> <%static:js group='application'/> diff --git a/lms/templates/wiki/preview_inline.html b/lms/templates/wiki/preview_inline.html index 1dd3ede9208c..4fdab7a5756f 100644 --- a/lms/templates/wiki/preview_inline.html +++ b/lms/templates/wiki/preview_inline.html @@ -40,7 +40,6 @@

{{ title }}

{% javascript 'application' %} - {% javascript 'module-js' %} {% with mathjax_mode='wiki' %} {% include "mathjax_include.html" %} {% endwith %} diff --git a/xmodule/assets/README.rst b/xmodule/assets/README.rst index c87642384f6d..618ba029246f 100644 --- a/xmodule/assets/README.rst +++ b/xmodule/assets/README.rst @@ -75,14 +75,11 @@ Currently, edx-platform XBlock JS is defined both here in `xmodule/assets`_ and * `VerticalBlock`_ * `LibrarySourcedBlock`_ -* Some XBlock JS is also processed through Django Pipeline and used in a couple specific legacy places. - As part of an `active build refactoring`_: * We update the older builtin XBlocks to reference their JS directly rather than using copies of it. * We will move ``webpack.xmodule.config.js`` here instead of generating it. * We will consolidate all edx-platform XBlock JS here in `xmodule/assets`_. -* We will remove XBlock JS from Django Pipeline. * We will delete the ``xmodule_assets`` script. .. _xmodule/assets: https://github.com/openedx/edx-platform/tree/master/xmodule/assets