edx-enterprise-data 8.12.0__tar.gz → 8.12.1__tar.gz

{edx_enterprise_data-8.12.0 → edx_enterprise_data-8.12.1}/CHANGELOG.rst

@@ -15,6 +15,10 @@ Unreleased
 ----------
 
 =========================
+[8.12.1] - 2024-09-16
+---------------------
+  * fix: Remove hyphens from enterprise customer UUID before database query.
+
 [8.12.0] - 2024-09-06
 ---------------------
   * refactor: Performance optimizations for enrollments related API endpoints.

{edx_enterprise_data-8.12.0/edx_enterprise_data.egg-info → edx_enterprise_data-8.12.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: edx-enterprise-data
-Version: 8.12.0
+Version: 8.12.1
 Summary: Enterprise Reporting
 Home-page: https://github.com/openedx/edx-enterprise-data
 Author: edX

{edx_enterprise_data-8.12.0 → edx_enterprise_data-8.12.1/edx_enterprise_data.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: edx-enterprise-data
-Version: 8.12.0
+Version: 8.12.1
 Summary: Enterprise Reporting
 Home-page: https://github.com/openedx/edx-enterprise-data
 Author: edX

{edx_enterprise_data-8.12.0 → edx_enterprise_data-8.12.1}/enterprise_data/__init__.py

@@ -2,4 +2,4 @@
 Enterprise data api application. This Django app exposes API endpoints used by enterprises.
 """
 
-__version__ = "8.12.0"
+__version__ = "8.12.1"

{edx_enterprise_data-8.12.0 → edx_enterprise_data-8.12.1}/enterprise_data/admin_analytics/database/queries/fact_enrollment_admin_dash.py

@@ -96,10 +96,8 @@ class FactEnrollmentAdminDashQueries:
         """
         # Some local environments raise error when course_title is added in SELECT without GROUP BY.
         # If you face this issue, you can remove course_title from SELECT.
-
-        # TODO: Re-add course_title after testing.
         return f"""
-            SELECT course_key, enroll_type, count(course_key) as enrollment_count
+            SELECT course_key, course_title , enroll_type, count(course_key) as enrollment_count
             FROM fact_enrollment_admin_dash
             WHERE enterprise_customer_uuid=%(enterprise_customer_uuid)s AND
                 enterprise_enrollment_date BETWEEN %(start_date)s AND %(end_date)s

{edx_enterprise_data-8.12.0 → edx_enterprise_data-8.12.1}/enterprise_data/api/v1/views/analytics_enrollments.py

@@ -43,6 +43,9 @@ class AdvanceAnalyticsEnrollmentsView(AnalyticsPaginationMixin, ViewSet):
         """
         Get individual enrollments data for the enterprise.
         """
+        # Remove hyphens from the UUID
+        enterprise_uuid = enterprise_uuid.replace('-', '')
+
         serializer = AdvanceAnalyticsQueryParamSerializer(data=request.GET)
         serializer.is_valid(raise_exception=True)
         min_enrollment_date, _ = FactEnrollmentAdminDashTable().get_enrollment_date_range(
@@ -122,6 +125,9 @@ class AdvanceAnalyticsEnrollmentsView(AnalyticsPaginationMixin, ViewSet):
         2. `top_courses_by_enrollments`: This will show the top courses by enrollments.
         3. `top_subjects_by_enrollments`: This will show the top subjects by enrollments.
         """
+        # Remove hyphens from the UUID
+        enterprise_uuid = enterprise_uuid.replace('-', '')
+
         serializer = AdvanceAnalyticsEnrollmentStatsSerializer(data=request.GET)
         serializer.is_valid(raise_exception=True)
 

{edx_enterprise_data-8.12.0 → edx_enterprise_data-8.12.1}/enterprise_data/api/v1/views/base.py

@@ -92,7 +92,7 @@ class AnalyticsPaginationMixin:
             (Response): The pagination data.
         """
         page_count = math.ceil(total_count / page_size)
-        if page <= 0 or page > page_count:
+        if page_count > 0 and (page <= 0 or page > page_count):
             raise NotFound('Invalid page.')
 
         return Response({