Merge pull request #46 from openedx/feanil/configurable_branch

feanil/configurable branch

Author: feanil

plugins/tutor-contrib-ltistore/tutor_ltistore/patches/openedx-cms-common-settings

@@ -0,0 +1,17 @@
+
+# Add the lti store filter to the filter list, creating the configuration if it doesn't
+# exist but updating it if it does exist.
+_filter_name = 'org.openedx.xblock.lti_consumer.configuration.listed.v1'
+_step_name = 'lti_store.pipelines.GetLtiConfigurations'
+
+try:
+    not OPEN_EDX_FILTERS_CONFIG
+except NameError:  # OPEN_EDX_FILTERS_CONFIG is not defined
+    OPEN_EDX_FILTERS_CONFIG = {}
+
+if not OPEN_EDX_FILTERS_CONFIG.get(_filter_name):
+    OPEN_EDX_FILTERS_CONFIG[_filter_name] = {
+        "fail_silently": False,
+        "pipeline": [],
+    }
+OPEN_EDX_FILTERS_CONFIG[_filter_name]["pipeline"].append(_step_name)

plugins/tutor-contrib-ltistore/tutor_ltistore/patches/openedx-common-settings

@@ -1,2 +1,2 @@
 RUN --mount=type=cache,target=/openedx/.cache/pip,sharing=shared \
-    pip install git+https://github.com/feanil/openedx-ltistore.git@feanil/lti1p3_fixes
+    pip install {{ LTISTORE_PIP_INSTALL }}

plugins/tutor-contrib-ltistore/tutor_ltistore/patches/openedx-dockerfile-post-python-requirements

@@ -17,6 +17,12 @@
         # Each new setting is a pair: (setting_name, default_value).
         # Prefix your setting names with 'LTISTORE_'.
         ("LTISTORE_VERSION", __version__),
+        # It's useful to change this to git sources during development
+        # so we're making this a config value you can change.
+        (
+            "LTISTORE_PIP_INSTALL",
+            "git+https://github.com/feanil/openedx-ltistore.git@feanil/minimal_lti1p3_fixes",
+        ),
     ]
 )