Merge pull request #2169 from openedx/hamzawaleed01/rename-group-name

chore: rename django group name
openedx · Jul 10, 2024 · 0758ddc · 0758ddc
2 parents c097dc4 + 303d69c
commit 0758ddc
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 4 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -17,6 +17,10 @@ Unreleased
 ----------
 * nothing unreleased
 
+[4.21.3]
+---------
+* chore: rename PAs django group name
+
 [4.21.2]
 ---------
 * feat: Updated EnterpriseCustomerSerializer to include additional fields

diff --git a/enterprise/__init__.py b/enterprise/__init__.py
@@ -2,4 +2,4 @@
 Your project description goes here.
 """
 
-__version__ = "4.21.2"
+__version__ = "4.21.3"
diff --git a/enterprise/api/v1/permissions.py b/enterprise/api/v1/permissions.py
@@ -24,7 +24,7 @@ class IsInProvisioningAdminGroup(permissions.BasePermission):
     """
     Grant access to those users only who are part of the license provisiioning django group
     """
-    ALLOWED_API_GROUPS = ['provisioning-admins-group']
+    ALLOWED_API_GROUPS = ['provisioning_admins_group']
     message = 'Access denied: You do not have the necessary permissions to access this.'
 
     def has_permission(self, request, view):

diff --git a/enterprise/api/v1/views/enterprise_customer.py b/enterprise/api/v1/views/enterprise_customer.py
@@ -57,7 +57,7 @@ class EnterpriseCustomerViewSet(EnterpriseReadWriteModelViewSet):
     filter_backends = EnterpriseReadWriteModelViewSet.filter_backends + (EnterpriseLinkedUserFilterBackend,)
     pagination_class = PaginationWithFeatureFlags
 
-    PROVISIONING_ADMINS_GROUP = 'provisioning-admins-group'
+    PROVISIONING_ADMINS_GROUP = 'provisioning_admins_group'
     USER_ID_FILTER = 'enterprise_customer_users__user_id'
     FIELDS = (
         'uuid', 'slug', 'name', 'active', 'site', 'enable_data_sharing_consent',

diff --git a/tests/test_enterprise/api/constants.py b/tests/test_enterprise/api/constants.py
@@ -12,4 +12,4 @@
     ('<EntitiesDescriptor xmlns="urn:oasis:names:tc:SAML:2.0:metadata"><EntityDescriptor entityID="https://example.com"></EntityDescriptor></EntitiesDescriptor>', "https://example.com")  # pylint: disable=line-too-long
 ]
 
-PROVISIONING_ADMINS_GROUP = "provisioning-admins-group"
+PROVISIONING_ADMINS_GROUP = "provisioning_admins_group"