feat: Add retrieve object_tags REST API (#577)

openedx/features/content_tagging/api.py

@@ -101,20 +101,16 @@ def get_taxonomies_for_org(
 
 
 def get_content_tags(
-    object_id: str, taxonomy: Taxonomy = None, valid_only=True
+    object_id: str, taxonomy_id: str = None
 ) -> Iterator[ContentObjectTag]:
     """
     Generates a list of content tags for a given object.
 
     Pass taxonomy to limit the returned object_tags to a specific taxonomy.
-
-    Pass valid_only=False when displaying tags to content authors, so they can see invalid tags too.
-    Invalid tags will (probably) be hidden from learners.
     """
     for object_tag in oel_tagging.get_object_tags(
         object_id=object_id,
-        taxonomy=taxonomy,
-        valid_only=valid_only,
+        taxonomy_id=taxonomy_id,
     ):
         yield ContentObjectTag.cast(object_tag)
 

openedx/features/content_tagging/rest_api/v1/urls.py

@@ -6,10 +6,13 @@
 
 from django.urls.conf import path, include
 
+from openedx_tagging.core.tagging.rest_api.v1 import views as oel_tagging_views
+
 from . import views
 
 router = DefaultRouter()
 router.register("taxonomies", views.TaxonomyOrgView, basename="taxonomy")
+router.register("object_tags", oel_tagging_views.ObjectTagView, basename="object_tag")
 
 urlpatterns = [
     path('', include(router.urls))

openedx/features/content_tagging/tests/test_api.py

@@ -189,14 +189,12 @@ def test_get_content_tags_valid_for_org(
         object_tag_attr,
     ):
         taxonomy_id = getattr(self, taxonomy_attr).id
-        taxonomy = api.get_taxonomy(taxonomy_id)
         object_tag = getattr(self, object_tag_attr)
-        with self.assertNumQueries(1):
+        with self.assertNumQueries(2):
             valid_tags = list(
                 api.get_content_tags(
-                    taxonomy=taxonomy,
+                    taxonomy_id=taxonomy_id,
                     object_id=object_tag.object_id,
-                    valid_only=True,
                 )
             )
         assert len(valid_tags) == 1
@@ -219,14 +217,12 @@ def test_get_content_tags_include_invalid(
         object_tag_attr,
     ):
         taxonomy_id = getattr(self, taxonomy_attr).id
-        taxonomy = api.get_taxonomy(taxonomy_id)
         object_tag = getattr(self, object_tag_attr)
-        with self.assertNumQueries(1):
+        with self.assertNumQueries(2):
             valid_tags = list(
                 api.get_content_tags(
-                    taxonomy=taxonomy,
+                    taxonomy_id=taxonomy_id,
                     object_id=object_tag.object_id,
-                    valid_only=False,
                 )
             )
         assert len(valid_tags) == 1

requirements/edx/base.txt

@@ -43,6 +43,7 @@ attrs==23.1.0
     #   lti-consumer-xblock
     #   openedx-blockstore
     #   openedx-events
+    #   openedx-learning
     #   referencing
 babel==2.11.0
     # via
@@ -98,6 +99,7 @@ celery==5.3.1
     #   edx-celeryutils
     #   edx-enterprise
     #   event-tracking
+    #   openedx-learning
 certifi==2023.7.22
     # via
     #   -r requirements/edx/paver.txt

requirements/edx/development.txt

@@ -92,6 +92,7 @@ attrs==23.1.0
     #   lti-consumer-xblock
     #   openedx-blockstore
     #   openedx-events
+    #   openedx-learning
     #   referencing
 babel==2.11.0
     # via
@@ -175,6 +176,7 @@ celery==5.3.1
     #   edx-celeryutils
     #   edx-enterprise
     #   event-tracking
+    #   openedx-learning
 certifi==2023.7.22
     # via
     #   -r requirements/edx/doc.txt
@@ -1299,7 +1301,7 @@ openedx-filters==1.5.0
     #   -r requirements/edx/doc.txt
     #   -r requirements/edx/testing.txt
     #   lti-consumer-xblock
-openedx-learning==0.1.2
+openedx-learning==0.1.5
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/doc.txt

requirements/edx/doc.txt

@@ -60,6 +60,7 @@ attrs==23.1.0
     #   lti-consumer-xblock
     #   openedx-blockstore
     #   openedx-events
+    #   openedx-learning
     #   referencing
 babel==2.11.0
     # via
@@ -125,6 +126,7 @@ celery==5.3.1
     #   edx-celeryutils
     #   edx-enterprise
     #   event-tracking
+    #   openedx-learning
 certifi==2023.7.22
     # via
     #   -r requirements/edx/base.txt

requirements/edx/testing.txt

@@ -66,6 +66,7 @@ attrs==23.1.0
     #   lti-consumer-xblock
     #   openedx-blockstore
     #   openedx-events
+    #   openedx-learning
     #   referencing
 babel==2.11.0
     # via
@@ -131,6 +132,7 @@ celery==5.3.1
     #   edx-celeryutils
     #   edx-enterprise
     #   event-tracking
+    #   openedx-learning
 certifi==2023.7.22
     # via
     #   -r requirements/edx/base.txt