Merge pull request #25 from openedx/iahmad/ENT-8117

feat: added debug logs for skill tagging filter

CHANGELOG.rst

@@ -16,6 +16,15 @@ Unreleased
 
 *
 
+[0.1.4] - 2024-01-22
+************************************************
+
+Changed
+=======
+
+* Added debug logs for filter runs
+
+
 [0.1.3] - 2023-09-27
 ************************************************
 

skill_tagging/__init__.py

@@ -2,7 +2,7 @@
 Django app plugin for fetching and verifying tags for xblock skills.
 """
 
-__version__ = '0.1.3'
+__version__ = '0.1.4'
 
 # pylint: disable=invalid-name
 default_app_config = 'skill_tagging.apps.SkillTaggingConfig'

skill_tagging/pipeline.py

@@ -85,9 +85,17 @@ def run_filter(self, block, fragment, context, view):  # pylint: disable=argumen
         # Check whether we need to run this filter and only call the API.
         if not self.should_run_filter():
             return {"block": block, "fragment": fragment, "context": context, "view": view}
+        logger.info(
+            "[Xblock-Skill-Tagging] Filter run for AddVerticalBlockSkillVerificationSection. Block-ID: %s",
+            str(block.scope_ids.usage_id.block_id)
+        )
         skills = self.fetch_related_skills(block)
         if not skills:
             return {"block": block, "fragment": fragment, "context": context, "view": view}
+        logger.info(
+            "[Xblock-Skill-Tagging] Skills found for AddVerticalBlockSkillVerificationSection. Block-ID: %s",
+            str(block.scope_ids.usage_id.block_id)
+        )
         usage_id = block.scope_ids.usage_id
         data = self.get_skill_context(usage_id, block, skills)
         html = resource_string("static/tagging.html")
@@ -122,9 +130,17 @@ def run_filter(self, block, context):  # pylint: disable=arguments-differ
         if usage_id.block_type != "video" or not self.should_run_filter():
             # avoid fetching skills for other xblocks
             return {"block": block, "context": context}
+        logger.info(
+            "[Xblock-Skill-Tagging] Filter run for AddVideoBlockSkillVerificationComponent. Block-ID: %s",
+            str(block.scope_ids.usage_id.block_id)
+        )
         skills = self.fetch_related_skills(block)
         if not skills:
             return {"block": block, "context": context}
+        logger.info(
+            "[Xblock-Skill-Tagging] Skills found for AddVideoBlockSkillVerificationComponent. Block-ID: %s",
+            str(block.scope_ids.usage_id.block_id)
+        )
         data = self.get_skill_context(usage_id, block, skills)
 
         def wrapper(fn):