Parcourir la source

Merge pull request #21683 from overleaf/msm-sso-admin-checkbox

[web] Change Group SSO radio in admin panel to checkbox

GitOrigin-RevId: c4a07400c6f399724be0368108be24ec6df67efa
Miguel Serrano il y a 1 an
Parent
commit
a5f38a8663

+ 2 - 4
services/web/frontend/js/features/subscription/components/dashboard/group-settings-button.tsx

@@ -10,10 +10,8 @@ export default function GroupSettingsButton({
   const { t } = useTranslation()
 
   const subscriptionHasManagedUsers =
-    subscription.features?.managedUsers !== false
-  const subscriptionHasGroupSSO =
-    subscription.features?.groupSSO === true ||
-    subscription.features?.groupSSO === null
+    subscription.features?.managedUsers === true
+  const subscriptionHasGroupSSO = subscription.features?.groupSSO === true
 
   let groupSettingRowSubText = ''
   if (subscriptionHasGroupSSO && subscriptionHasManagedUsers) {

+ 0 - 19
services/web/test/frontend/features/subscription/components/dashboard/managed-group-subscriptions.test.tsx

@@ -83,8 +83,6 @@ const managedGroupSubscriptions3: ManagedGroupSubscription[] =
   getManagedGroupSubscriptions(true, false)
 const managedGroupSubscriptions4: ManagedGroupSubscription[] =
   getManagedGroupSubscriptions(false, true)
-const managedGroupSubscriptions5: ManagedGroupSubscription[] =
-  getManagedGroupSubscriptions(null, true)
 
 describe('<ManagedGroupSubscriptions />', function () {
   afterEach(function () {
@@ -196,23 +194,6 @@ describe('<ManagedGroupSubscriptions />', function () {
     await screen.findAllByText('Configure and manage SSO and Managed Users')
   })
 
-  it('renders Group SSO settings when the feature is null', async function () {
-    renderWithSubscriptionDashContext(<ManagedGroupSubscriptions />, {
-      metaTags: [
-        {
-          name: 'ol-managedGroupSubscriptions',
-          value: managedGroupSubscriptions5,
-        },
-        {
-          name: 'ol-groupSettingsEnabledFor',
-          value: [managedGroupSubscriptions5[0]._id],
-        },
-      ],
-    })
-    await screen.findAllByText('Manage group settings')
-    await screen.findAllByText('Configure and manage SSO and Managed Users')
-  })
-
   it('does not render Group SSO settings when the feature is turned off', async function () {
     renderWithSubscriptionDashContext(<ManagedGroupSubscriptions />, {
       metaTags: [