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

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

GitOrigin-RevId: c4a07400c6f399724be0368108be24ec6df67efa
This commit is contained in:
Miguel Serrano
2024-11-20 12:28:20 +01:00
committed by Copybot
parent 2d122323eb
commit d080218d96
2 changed files with 2 additions and 23 deletions

View File

@@ -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) {

View File

@@ -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: [