Merge branch 'master' into bilalqamar95/node-v20-upgrade

CHANGELOG.rst

@@ -17,6 +17,14 @@ Unreleased
 ----------
 * nothing unreleased
 
+[4.25.19]
+---------
+* feat: remove logging to debug SAP SuccessFactors transmission issues
+
+[4.25.18]
+---------
+* chore: requirement updates
+
 [4.25.17]
 ---------
 * feat: add pagination to the support tool customer list

enterprise/__init__.py

@@ -2,4 +2,4 @@
 Your project description goes here.
 """
 
-__version__ = "4.25.17"
+__version__ = "4.25.19"

integrated_channels/integrated_channel/exporters/learner_data.py

@@ -124,15 +124,6 @@ def bulk_assessment_level_export(self):
         # Create a record of each subsection from every enterprise enrollment
         for enterprise_enrollment in enrollment_queryset:
             if not LearnerExporter.has_data_sharing_consent(enterprise_enrollment):
-                # Adding logging to debug the issue we are having with a customer using SAPSF channel
-                LOGGER.info(generate_formatted_log(
-                    self.enterprise_configuration.channel_code(),
-                    self.enterprise_configuration.enterprise_customer.uuid,
-                    None,
-                    None,
-                    f'[SAPSF] Transmission skipped for {enterprise_enrollment.enterprise_customer_user.user_id}'
-                    'due to missing data sharing consent.'
-                ))
                 continue
 
             assessment_grade_data = self._collect_assessment_grades_data(enterprise_enrollment)
@@ -201,16 +192,6 @@ def single_assessment_level_export(self, **kwargs):
             detect_grade_updated=self.INCLUDE_GRADE_FOR_COMPLETION_AUDIT_CHECK,
         )
 
-        # Adding logging to debug the issue we are having with a customer using SAPSF channel
-        LOGGER.info(generate_formatted_log(     # pragma: no cover
-            self.enterprise_configuration.channel_code(),
-            self.enterprise_configuration.enterprise_customer.uuid,
-            None,
-            None,
-            f'[SAPSF] Transmission already sent for {enterprise_enrollment.enterprise_customer_user.user_id}'
-            f'is_already_transmitted returned {already_transmitted}'
-        ))
-
         if not (TransmissionAudit and already_transmitted) and LearnerExporter.has_data_sharing_consent(
                 enterprise_enrollment):
 
@@ -221,14 +202,6 @@ def single_assessment_level_export(self, **kwargs):
                 assessment_grade_data=assessment_grade_data,
             )
             if records:
-                # Adding logging to debug the issue we are having with a customer using SAPSF channel
-                LOGGER.info(generate_formatted_log(     # pragma: no cover
-                    self.enterprise_configuration.channel_code(),
-                    self.enterprise_configuration.enterprise_customer.uuid,
-                    None,
-                    None,
-                    f'[SAPSF] Transmission sent for {enterprise_enrollment.enterprise_customer_user.user_id}'
-                ))
                 # There are some cases where we won't receive a record from the above
                 # method; right now, that should only happen if we have an Enterprise-linked
                 # user for the integrated channel, and transmission of that user's
@@ -524,15 +497,6 @@ def _filter_out_pre_transmitted_enrollments(
                         grade,
                         detect_grade_updated=self.INCLUDE_GRADE_FOR_COMPLETION_AUDIT_CHECK,
                     ):
-                # Adding logging to debug the issue we are having with a customer using SAPSF channel
-                LOGGER.info(generate_formatted_log(
-                    self.enterprise_configuration.channel_code(),
-                    self.enterprise_configuration.enterprise_customer.uuid,
-                    None,
-                    None,
-                    f'[SAPSF] Transmission skipped for {enterprise_enrollment.enterprise_customer_user.user_id}'
-                    'transmission_audit and already_transmitted returned True.'
-                ))
                 # We've already sent a completion status for this enrollment
                 LOGGER.info(generate_formatted_log(
                     channel_name, enterprise_customer_uuid, lms_user_id, course_id,

pylintrc

@@ -292,6 +292,7 @@ disable =
 	no-member,
 	missing-timeout,
 	use-yield-from,
+	too-many-positional-arguments,
 
 [REPORTS]
 output-format = text
@@ -388,4 +389,4 @@ int-import-graph =
 [EXCEPTIONS]
 overgeneral-exceptions = builtins.Exception
 
-# 126dc53d1644a269339811dea1e20f5bd1bf3264
+# 54077c76e994996b5dcfc894527265ff056e760f

pylintrc_tweaks

@@ -10,3 +10,4 @@ disable+ =
     no-member,
     missing-timeout,
     use-yield-from,
+    too-many-positional-arguments,

requirements/celery53.txt

@@ -1,9 +1,9 @@
 amqp==5.2.0
-billiard==4.2.0
+billiard==4.2.1
 celery==5.4.0
 click==8.1.7
 click-didyoumean==0.3.1
 click-repl==0.3.0
-kombu==5.4.1
+kombu==5.4.2
 prompt-toolkit==3.0.47
 vine==5.1.0

requirements/ci.txt

@@ -6,13 +6,13 @@
 #
 distlib==0.3.8
     # via virtualenv
-filelock==3.16.0
+filelock==3.16.1
     # via
     #   tox
     #   virtualenv
 packaging==24.1
     # via tox
-platformdirs==4.3.3
+platformdirs==4.3.6
     # via virtualenv
 pluggy==1.5.0
     # via tox
@@ -24,5 +24,5 @@ tox==3.28.0
     # via
     #   -c requirements/constraints.txt
     #   -r requirements/ci.in
-virtualenv==20.26.4
+virtualenv==20.26.5
     # via tox

requirements/dev.txt

@@ -55,7 +55,7 @@ asn1crypto==1.5.1
     #   -r requirements/test-master.txt
     #   -r requirements/test.txt
     #   snowflake-connector-python
-astroid==3.2.4
+astroid==3.3.3
     # via
     #   pylint
     #   pylint-celery
@@ -82,7 +82,7 @@ beautifulsoup4==4.12.3
     # via
     #   -r requirements/doc.txt
     #   pydata-sphinx-theme
-billiard==4.2.0
+billiard==4.2.1
     # via
     #   -r requirements/doc.txt
     #   -r requirements/test-master.txt
@@ -408,7 +408,7 @@ factory-boy==3.3.1
     #   -c requirements/constraints.txt
     #   -r requirements/doc.txt
     #   -r requirements/test.txt
-faker==28.4.1
+faker==29.0.0
     # via
     #   -r requirements/doc.txt
     #   -r requirements/test.txt
@@ -489,7 +489,7 @@ jwcrypto==1.5.6
     #   -r requirements/test-master.txt
     #   -r requirements/test.txt
     #   django-oauth-toolkit
-kombu==5.4.1
+kombu==5.4.2
     # via
     #   -r requirements/doc.txt
     #   -r requirements/test-master.txt
@@ -669,7 +669,7 @@ pyjwt[crypto]==2.9.0
     #   edx-drf-extensions
     #   edx-rest-api-client
     #   snowflake-connector-python
-pylint==3.2.7
+pylint==3.3.0
     # via
     #   edx-lint
     #   pylint-celery
@@ -932,6 +932,7 @@ tzdata==2024.1
     #   -r requirements/test-master.txt
     #   -r requirements/test.txt
     #   celery
+    #   kombu
 unicodecsv==0.14.1
     # via
     #   -r requirements/doc.txt
@@ -957,7 +958,7 @@ vine==5.1.0
     #   amqp
     #   celery
     #   kombu
-virtualenv==20.26.4
+virtualenv==20.26.5
     # via tox
 wcwidth==0.2.13
     # via

requirements/doc.txt

@@ -53,7 +53,7 @@ bcrypt==4.2.0
     #   paramiko
 beautifulsoup4==4.12.3
     # via pydata-sphinx-theme
-billiard==4.2.0
+billiard==4.2.1
     # via
     #   -r requirements/test-master.txt
     #   celery
@@ -248,7 +248,7 @@ factory-boy==3.3.1
     # via
     #   -c requirements/constraints.txt
     #   -r requirements/doc.in
-faker==28.4.1
+faker==29.0.0
     # via factory-boy
 fastavro==1.9.5
     # via
@@ -290,7 +290,7 @@ jwcrypto==1.5.6
     # via
     #   -r requirements/test-master.txt
     #   django-oauth-toolkit
-kombu==5.4.1
+kombu==5.4.2
     # via
     #   -r requirements/test-master.txt
     #   celery
@@ -523,6 +523,7 @@ tzdata==2024.1
     # via
     #   -r requirements/test-master.txt
     #   celery
+    #   kombu
 unicodecsv==0.14.1
     # via -r requirements/test-master.txt
 uritemplate==4.1.1

requirements/edx-platform-constraints.txt

@@ -455,7 +455,7 @@ edx-drf-extensions==10.3.0
     #   edx-when
     #   edxval
     #   openedx-learning
-edx-enterprise==4.25.11
+edx-enterprise==4.25.13
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/kernel.in
@@ -470,10 +470,8 @@ edx-i18n-tools==1.5.0
     #   ora2
 edx-milestones==0.6.0
     # via -r requirements/edx/kernel.in
-edx-name-affirmation==2.4.0
-    # via
-    #   -c requirements/edx/../constraints.txt
-    #   -r requirements/edx/kernel.in
+edx-name-affirmation==2.4.1
+    # via -r requirements/edx/kernel.in
 edx-opaque-keys[django]==2.11.0
     # via
     #   -r requirements/edx/kernel.in
@@ -810,7 +808,7 @@ openedx-filters==1.9.0
     #   -r requirements/edx/kernel.in
     #   lti-consumer-xblock
     #   ora2
-openedx-learning==0.11.4
+openedx-learning==0.11.5
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/kernel.in
@@ -820,7 +818,7 @@ optimizely-sdk==4.1.1
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/bundled.in
-ora2==6.11.2
+ora2==6.12.0
     # via -r requirements/edx/bundled.in
 packaging==24.1
     # via

requirements/js_test.txt

@@ -67,7 +67,7 @@ python-dateutil==2.9.0.post0
     # via tempora
 pyyaml==6.0.2
     # via jasmine
-selenium==4.24.0
+selenium==4.25.0
     # via jasmine
 six==1.16.0
     # via python-dateutil

requirements/test-master.txt

@@ -40,7 +40,7 @@ bcrypt==4.2.0
     # via
     #   -c requirements/edx-platform-constraints.txt
     #   paramiko
-billiard==4.2.0
+billiard==4.2.1
     # via celery
 bleach==6.1.0
     # via
@@ -297,7 +297,7 @@ jwcrypto==1.5.6
     # via
     #   -c requirements/edx-platform-constraints.txt
     #   django-oauth-toolkit
-kombu==5.4.1
+kombu==5.4.2
     # via celery
 markupsafe==2.1.5
     # via
@@ -494,6 +494,7 @@ tzdata==2024.1
     # via
     #   -c requirements/edx-platform-constraints.txt
     #   celery
+    #   kombu
 unicodecsv==0.14.1
     # via
     #   -c requirements/edx-platform-constraints.txt

requirements/test.txt

@@ -230,7 +230,7 @@ factory-boy==3.3.1
     # via
     #   -c requirements/constraints.txt
     #   -r requirements/test.in
-faker==28.4.1
+faker==29.0.0
     # via factory-boy
 fastavro==1.9.5
     # via
@@ -486,6 +486,7 @@ tzdata==2024.1
     # via
     #   -r requirements/test-master.txt
     #   celery
+    #   kombu
 unicodecsv==0.14.1
     # via -r requirements/test-master.txt
 uritemplate==4.1.1