fix: use openedx-learning branch from pr#157

which reduces the query count. TO DO: once PR is merged and tagged, update the requirement here.
open-craft · Feb 7, 2024 · af6e41d · af6e41d
1 parent 9fd3c75
commit af6e41d
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/openedx/core/djangoapps/content_tagging/rest_api/v1/tests/test_views.py b/openedx/core/djangoapps/content_tagging/rest_api/v1/tests/test_views.py
@@ -497,7 +497,7 @@ def test_list_taxonomy_query_count(self):
         url = TAXONOMY_ORG_LIST_URL + f'?org={self.orgA.short_name}&enabled=true'
 
         self.client.force_authenticate(user=self.staff)
-        with self.assertNumQueries(17):  # TODO Why so many queries?
+        with self.assertNumQueries(15):  # TODO Why so many queries?
             response = self.client.get(url)
 
         assert response.status_code == 200

diff --git a/requirements/constraints.txt b/requirements/constraints.txt
@@ -108,7 +108,7 @@ libsass==0.10.0
 click==8.1.6
 
 # pinning this version to avoid updates while the library is being developed
-openedx-learning==0.5.1
+openedx-learning @ git+https://github.com/open-craft/openedx-learning.git@jill/tagging-less-queries
 
 # Open AI version 1.0.0 dropped support for openai.ChatCompletion which is currently in use in enterprise.
 openai<=0.28.1

diff --git a/requirements/edx/base.txt b/requirements/edx/base.txt
@@ -782,7 +782,7 @@ openedx-filters==1.6.0
     # via
     #   -r requirements/edx/kernel.in
     #   lti-consumer-xblock
-openedx-learning==0.5.1
+openedx-learning @ git+https://github.com/open-craft/openedx-learning.git@jill/tagging-less-queries
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/kernel.in

diff --git a/requirements/edx/development.txt b/requirements/edx/development.txt
@@ -1313,7 +1313,7 @@ openedx-filters==1.6.0
     #   -r requirements/edx/doc.txt
     #   -r requirements/edx/testing.txt
     #   lti-consumer-xblock
-openedx-learning==0.5.1
+openedx-learning @ git+https://github.com/open-craft/openedx-learning.git@jill/tagging-less-queries
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/doc.txt

diff --git a/requirements/edx/doc.txt b/requirements/edx/doc.txt
@@ -924,7 +924,7 @@ openedx-filters==1.6.0
     # via
     #   -r requirements/edx/base.txt
     #   lti-consumer-xblock
-openedx-learning==0.5.1
+openedx-learning @ git+https://github.com/open-craft/openedx-learning.git@jill/tagging-less-queries
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/base.txt

diff --git a/requirements/edx/testing.txt b/requirements/edx/testing.txt
@@ -982,7 +982,7 @@ openedx-filters==1.6.0
     # via
     #   -r requirements/edx/base.txt
     #   lti-consumer-xblock
-openedx-learning==0.5.1
+openedx-learning @ git+https://github.com/open-craft/openedx-learning.git@jill/tagging-less-queries
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/base.txt