Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: 🐛 ENT-4525 fix learner portal experience when missing courseEnrollment records (#27611)

Browse Source 
* fix: 🐛 ENT-4525 fix learner portal experience courseEnrollment records are missing

sometimes integrations may create EnterpriseCourseEnrollments but for some reason missing courseEnrollment, in this case we want to just filter out the EnterpriseCourseEnrollments instead of erroring the learner poratl

ENT-4525

* fix: pylint fixes

pylint fixes
This commit is contained in:
Binod Pant
2021-05-13 07:27:17 -04:00
committed by GitHub
parent cd9fed2557
commit da9afe6290
6 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
lms/djangoapps/certificates/management/commands/tests/test_cert_management.py
View File

@@ -7,7 +7,6 @@ import ddt
from django.core.management import call_command
from django.core.management.base import CommandError
from django.test.utils import override_settings
from opaque_keys.edx.locator import CourseLocator
from common.djangoapps.course_modes.models import CourseMode
from common.djangoapps.student.tests.factories import CourseEnrollmentFactory, UserFactory

2
lms/djangoapps/certificates/queue.py
View File

@@ -175,7 +175,7 @@ class XQueueCertInterface:
raise NotImplementedError
# pylint: disable=too-many-statements
def add_cert(self, student, course_id, course=None, forced_grade=None, template_file=None, generate_pdf=True):
def add_cert(self, student, course_id, course=None, forced_grade=None, template_file=None, generate_pdf=True): # lint-amnesty, pylint: disable=unused-argument
"""
Request a new certificate for a student.

2
requirements/constraints.txt
View File

@@ -39,7 +39,7 @@ django-storages<1.9
# The team that owns this package will manually bump this package rather than having it pulled in automatically.
# This is to allow them to better control its deployment and to do it in a process that works better
# for them.
edx-enterprise==3.22.16
edx-enterprise==3.23.1
# Newer versions need a more recent version of python-dateutil
freezegun==0.3.12

2
requirements/edx/base.txt
View File

@@ -101,7 +101,7 @@ edx-django-release-util==1.0.0 # via -r requirements/edx/base.in
edx-django-sites-extensions==3.0.0 # via -r requirements/edx/base.in
edx-django-utils==4.0.0 # via -r requirements/edx/base.in, django-config-models, edx-drf-extensions, edx-enterprise, edx-rest-api-client, edx-toggles, edx-when, ora2, super-csv
edx-drf-extensions==6.5.0 # via -r requirements/edx/base.in, edx-completion, edx-enterprise, edx-organizations, edx-proctoring, edx-rbac, edx-when, edxval
edx-enterprise==3.22.16 # via -c requirements/edx/../constraints.txt, -r requirements/edx/base.in
edx-enterprise==3.23.1 # via -c requirements/edx/../constraints.txt, -r requirements/edx/base.in
edx-event-routing-backends==4.0.1 # via -r requirements/edx/base.in
edx-i18n-tools==0.5.3 # via ora2
edx-milestones==0.3.1 # via -r requirements/edx/base.in

2
requirements/edx/development.txt
View File

@@ -111,7 +111,7 @@ edx-django-release-util==1.0.0 # via -r requirements/edx/testing.txt
edx-django-sites-extensions==3.0.0 # via -r requirements/edx/testing.txt
edx-django-utils==4.0.0 # via -r requirements/edx/testing.txt, django-config-models, edx-drf-extensions, edx-enterprise, edx-rest-api-client, edx-toggles, edx-when, ora2, super-csv
edx-drf-extensions==6.5.0 # via -r requirements/edx/testing.txt, edx-completion, edx-enterprise, edx-organizations, edx-proctoring, edx-rbac, edx-when, edxval
edx-enterprise==3.22.16 # via -c requirements/edx/../constraints.txt, -r requirements/edx/testing.txt
edx-enterprise==3.23.1 # via -c requirements/edx/../constraints.txt, -r requirements/edx/testing.txt
edx-event-routing-backends==4.0.1 # via -r requirements/edx/testing.txt
edx-i18n-tools==0.5.3 # via -r requirements/edx/testing.txt, ora2
edx-lint==5.0.0 # via -r requirements/edx/testing.txt

2
requirements/edx/testing.txt
View File

@@ -108,7 +108,7 @@ edx-django-release-util==1.0.0 # via -r requirements/edx/base.txt
edx-django-sites-extensions==3.0.0 # via -r requirements/edx/base.txt
edx-django-utils==4.0.0 # via -r requirements/edx/base.txt, django-config-models, edx-drf-extensions, edx-enterprise, edx-rest-api-client, edx-toggles, edx-when, ora2, super-csv
edx-drf-extensions==6.5.0 # via -r requirements/edx/base.txt, edx-completion, edx-enterprise, edx-organizations, edx-proctoring, edx-rbac, edx-when, edxval
edx-enterprise==3.22.16 # via -c requirements/edx/../constraints.txt, -r requirements/edx/base.txt
edx-enterprise==3.23.1 # via -c requirements/edx/../constraints.txt, -r requirements/edx/base.txt
edx-event-routing-backends==4.0.1 # via -r requirements/edx/base.txt
edx-i18n-tools==0.5.3 # via -r requirements/edx/base.txt, -r requirements/edx/testing.in, ora2
edx-lint==5.0.0 # via -r requirements/edx/testing.in