Skip to content

Commit 7c86626

Browse files
authored
Merge pull request #38209 from openedx/pwnage101/ENT-11663
feat: remove enterprise and consent from OPTIONAL_APPS
2 parents 847de8f + d890c7a commit 7c86626

7 files changed

Lines changed: 9 additions & 9 deletions

File tree

openedx/envs/common.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -775,9 +775,8 @@ def make_mako_module_dir(settings):
775775
# edxval
776776
('edxval', 'openedx.core.djangoapps.content.course_overviews.apps.CourseOverviewsConfig'),
777777

778-
# Enterprise Apps (http://github.com/openedx/edx-enterprise)
779-
('enterprise', None),
780-
('consent', None),
778+
# Deprecated apps from the edx-enterprise package. We're working on removing these as part of
779+
# pluginifying edx-enterprise (<https://discuss.openedx.org/t/18316>)
781780
('integrated_channels.integrated_channel', None),
782781
('integrated_channels.degreed', None),
783782
('integrated_channels.degreed2', None),

openedx/features/enterprise_support/api.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
import requests
1111
from crum import get_current_request
12+
from django.apps import apps as django_apps
1213
from django.conf import settings
1314
from django.contrib.auth.models import User # lint-amnesty, pylint: disable=imported-auth-user
1415
from django.contrib.sites.models import Site
@@ -373,7 +374,7 @@ def enterprise_enabled():
373374
"""
374375
Determines whether the Enterprise app is installed
375376
"""
376-
return 'enterprise' in settings.INSTALLED_APPS and settings.FEATURES.get('ENABLE_ENTERPRISE_INTEGRATION', False)
377+
return django_apps.is_installed('enterprise') and settings.FEATURES.get('ENABLE_ENTERPRISE_INTEGRATION', False)
377378

378379

379380
def enterprise_is_enabled(otherwise=None):

requirements/constraints.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ django-stubs<6
4444
# The team that owns this package will manually bump this package rather than having it pulled in automatically.
4545
# This is to allow them to better control its deployment and to do it in a process that works better
4646
# for them.
47-
edx-enterprise==6.7.0
47+
edx-enterprise==6.8.1
4848

4949
# Date: 2023-07-26
5050
# Our legacy Sass code is incompatible with anything except this ancient libsass version.

requirements/edx/base.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -478,7 +478,7 @@ edx-drf-extensions==10.6.0
478478
# enterprise-integrated-channels
479479
# openedx-authz
480480
# openedx-core
481-
edx-enterprise==6.7.0
481+
edx-enterprise==6.8.1
482482
# via
483483
# -c requirements/constraints.txt
484484
# -r requirements/edx/kernel.in

requirements/edx/development.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -758,7 +758,7 @@ edx-drf-extensions==10.6.0
758758
# enterprise-integrated-channels
759759
# openedx-authz
760760
# openedx-core
761-
edx-enterprise==6.7.0
761+
edx-enterprise==6.8.1
762762
# via
763763
# -c requirements/constraints.txt
764764
# -r requirements/edx/doc.txt

requirements/edx/doc.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -569,7 +569,7 @@ edx-drf-extensions==10.6.0
569569
# enterprise-integrated-channels
570570
# openedx-authz
571571
# openedx-core
572-
edx-enterprise==6.7.0
572+
edx-enterprise==6.8.1
573573
# via
574574
# -c requirements/constraints.txt
575575
# -r requirements/edx/base.txt

requirements/edx/testing.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -589,7 +589,7 @@ edx-drf-extensions==10.6.0
589589
# enterprise-integrated-channels
590590
# openedx-authz
591591
# openedx-core
592-
edx-enterprise==6.7.0
592+
edx-enterprise==6.8.1
593593
# via
594594
# -c requirements/constraints.txt
595595
# -r requirements/edx/base.txt

0 commit comments

Comments
 (0)