fix: don't trigger celery task if giturl is missing

fix: default git export directory path updated

Author: marslanabdulrauf

src/ol_openedx_git_auto_export/BUILD

@@ -12,7 +12,7 @@ python_distribution(
     dependencies=[":git_auto_export"],
     provides=setup_py(
         name="ol-openedx-git-auto-export",
-        version="0.3.1",
+        version="0.3.2",
         license="BSD-3-Clause",
         description="A plugin that auto saves the course OLX to git when an author publishes it",
         entry_points={

src/ol_openedx_git_auto_export/settings/common.py

@@ -5,5 +5,5 @@
 
 def plugin_settings(settings):
     """Settings for the git auto export plugin."""  # noqa: D401
-    settings.GIT_REPO_EXPORT_DIR = "/edx/var/edxapp/export_course_repos"
+    settings.GIT_REPO_EXPORT_DIR = "/openedx/export_course_repos"
     settings.FEATURES[ENABLE_GIT_AUTO_EXPORT] = True

src/ol_openedx_git_auto_export/settings/production.py

@@ -5,5 +5,5 @@
 
 def plugin_settings(settings):
     """Settings for the git auto export plugin."""  # noqa: D401
-    settings.GIT_REPO_EXPORT_DIR = "/edx/var/edxapp/export_course_repos"
+    settings.GIT_REPO_EXPORT_DIR = "/openedx/export_course_repos"
     settings.FEATURES[ENABLE_GIT_AUTO_EXPORT] = True

src/ol_openedx_git_auto_export/signals.py

@@ -5,7 +5,7 @@
 from django.dispatch import receiver
 from ol_openedx_git_auto_export.constants import ENABLE_GIT_AUTO_EXPORT
 from ol_openedx_git_auto_export.tasks import async_export_to_git
-from xmodule.modulestore.django import SignalHandler
+from xmodule.modulestore.django import SignalHandler, modulestore
 
 log = logging.getLogger(__name__)
 
@@ -20,7 +20,7 @@ def listen_for_course_publish(
     Receives publishing signal and performs publishing related workflows
     """
     git_repo_export_dir = getattr(
-        settings, "GIT_REPO_EXPORT_DIR", "/edx/var/edxapp/export_course_repos"
+        settings, "GIT_REPO_EXPORT_DIR", "/openedx/export_course_repos"
     )
     if not os.path.exists(git_repo_export_dir):  # noqa: PTH110
         # for development/docker/vagrant if GIT_REPO_EXPORT_DIR folder does not exist then create it  # noqa: E501
@@ -32,6 +32,13 @@ def listen_for_course_publish(
     if settings.FEATURES.get("ENABLE_EXPORT_GIT") and settings.FEATURES.get(
         ENABLE_GIT_AUTO_EXPORT
     ):
+        course_module = modulestore().get_course(course_key)
+        if not course_module.giturl:
+            log.info(
+                "Course %s does not have a GIT URL set in course advance settings, skipping export.",  # noqa: E501
+                course_module.id,
+            )
+            return
         # If the Git auto-export is enabled, push the course changes to Git
         log.info(
             "Course published with auto-export enabled. Starting export... (course id: %s)",  # noqa: E501

src/ol_openedx_git_auto_export/tasks.py

@@ -15,13 +15,6 @@ def async_export_to_git(course_key_string, user=None):
     course_key = CourseKey.from_string(course_key_string)
     course_module = modulestore().get_course(course_key)
 
-    if course_module.giturl is None:
-        LOGGER.debug(
-            "Course %s does not have a giturl, skipping export.",
-            course_module.id,
-        )
-        return
-
     try:
         LOGGER.debug(
             "Starting async course content export to git (course id: %s)",