feat: remove enterprise and consent from OPTIONAL_APPS

- Remove ('enterprise', None) and ('consent', None) from OPTIONAL_APPS in
  openedx/envs/common.py. Now that edx-enterprise registers enterprise and
  consent as openedx LMS plugins via entry_points, get_plugin_apps() adds
  them to INSTALLED_APPS automatically. Keeping them in OPTIONAL_APPS would
  cause duplicate app label errors.
- Update enterprise_enabled() in enterprise_support/api.py to use
  django_apps.is_installed('enterprise') instead of the raw string check
  'enterprise' in settings.INSTALLED_APPS. The new plugin entry_point-based
  registration uses 'enterprise.apps.EnterpriseConfig' (not 'enterprise') in
  INSTALLED_APPS, so the raw string check no longer works.

ENT-11663

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: pwnage101

openedx/envs/common.py

@@ -774,9 +774,8 @@ def make_mako_module_dir(settings):
     # edxval
     ('edxval', 'openedx.core.djangoapps.content.course_overviews.apps.CourseOverviewsConfig'),
 
-    # Enterprise Apps (http://github.com/openedx/edx-enterprise)
-    ('enterprise', None),
-    ('consent', None),
+    # Deprecated apps from the edx-enterprise package. We're working on removing these as part of
+    # pluginifying edx-enterprise (<https://discuss.openedx.org/t/18316>)
     ('integrated_channels.integrated_channel', None),
     ('integrated_channels.degreed', None),
     ('integrated_channels.degreed2', None),

openedx/features/enterprise_support/api.py

@@ -9,6 +9,7 @@
 
 import requests
 from crum import get_current_request
+from django.apps import apps as django_apps
 from django.conf import settings
 from django.contrib.auth.models import User  # lint-amnesty, pylint: disable=imported-auth-user
 from django.contrib.sites.models import Site
@@ -373,7 +374,7 @@ def enterprise_enabled():
     """
     Determines whether the Enterprise app is installed
     """
-    return 'enterprise' in settings.INSTALLED_APPS and settings.FEATURES.get('ENABLE_ENTERPRISE_INTEGRATION', False)
+    return django_apps.is_installed('enterprise') and settings.FEATURES.get('ENABLE_ENTERPRISE_INTEGRATION', False)
 
 
 def enterprise_is_enabled(otherwise=None):

requirements/constraints.txt

@@ -44,7 +44,7 @@ django-stubs<6
 # 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==6.7.0
+edx-enterprise==6.8.0
 
 # Date: 2023-07-26
 # Our legacy Sass code is incompatible with anything except this ancient libsass version.

requirements/edx/base.txt

@@ -478,7 +478,7 @@ edx-drf-extensions==10.6.0
     #   enterprise-integrated-channels
     #   openedx-authz
     #   openedx-core
-edx-enterprise==6.7.0
+edx-enterprise==6.8.0
     # via
     #   -c requirements/constraints.txt
     #   -r requirements/edx/kernel.in

requirements/edx/development.txt

@@ -758,7 +758,7 @@ edx-drf-extensions==10.6.0
     #   enterprise-integrated-channels
     #   openedx-authz
     #   openedx-core
-edx-enterprise==6.7.0
+edx-enterprise==6.8.0
     # via
     #   -c requirements/constraints.txt
     #   -r requirements/edx/doc.txt

requirements/edx/doc.txt

@@ -569,7 +569,7 @@ edx-drf-extensions==10.6.0
     #   enterprise-integrated-channels
     #   openedx-authz
     #   openedx-core
-edx-enterprise==6.7.0
+edx-enterprise==6.8.0
     # via
     #   -c requirements/constraints.txt
     #   -r requirements/edx/base.txt

requirements/edx/testing.txt

@@ -589,7 +589,7 @@ edx-drf-extensions==10.6.0
     #   enterprise-integrated-channels
     #   openedx-authz
     #   openedx-core
-edx-enterprise==6.7.0
+edx-enterprise==6.8.0
     # via
     #   -c requirements/constraints.txt
     #   -r requirements/edx/base.txt