Skip to content

Commit 597dc3a

Browse files
committed
[PLAT-15464]Turn ON/OFF the runtime configs as per the LDAP Configure membership lookup settings
Summary: Refactor LDAPAuthRedesigned component and LDAPValidationSchema - Remove unused imports and icons - Update validation schema conditions for ldap_group_use_query and ldap_group_use_role_mapping Test Plan: Tested manually Reviewers: lsangappa Reviewed By: lsangappa Subscribers: yugaware Differential Revision: https://phorge.dev.yugabyte.com/D39333
1 parent 3d89269 commit 597dc3a

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

managed/ui/src/redesign/features/userAuth/ldap/LDAPAuthRedesigned.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,7 @@ import { escapeStr, TOAST_OPTIONS, UserDefaultRoleOptions } from '../UserAuthUti
5656
import { DisableAuthProviderModal } from '../DisableAuthProvider';
5757
import { transformData } from './LDAPUtils';
5858
import { getLDAPValidationSchema } from './LDAPValidationSchema';
59-
import { ArrowDropDown } from '@material-ui/icons';
6059
import { ReactComponent as User } from '../../../../redesign/assets/user-outline.svg';
61-
import { ReactComponent as UserGroupsIcon } from '../../../../redesign/assets/user-group.svg';
6260
import { ReactComponent as BulbIcon } from '../../../../redesign/assets/bulb.svg';
6361

6462
const useStyles = makeStyles((theme) => ({

managed/ui/src/redesign/features/userAuth/ldap/LDAPValidationSchema.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ export const getLDAPValidationSchema = (t: TFunction) => {
3434
['ldap_group_use_query', 'ldap_group_use_role_mapping'],
3535
{
3636
is: (ldap_group_use_query, ldap_group_use_role_mapping) =>
37-
ldap_group_use_role_mapping === true && ldap_group_use_query === 'true',
37+
ldap_group_use_role_mapping === true && ldap_group_use_query === 'false',
3838
then: Yup.string().required(t('messages.userAttribute')),
3939
otherwise: Yup.string()
4040
}
@@ -43,7 +43,7 @@ export const getLDAPValidationSchema = (t: TFunction) => {
4343
['ldap_group_use_query', 'ldap_group_use_role_mapping'],
4444
{
4545
is: (ldap_group_use_query, ldap_group_use_role_mapping) =>
46-
ldap_group_use_role_mapping === true && ldap_group_use_query === 'false',
46+
ldap_group_use_role_mapping === true && ldap_group_use_query === 'true',
4747
then: Yup.string().required(t('messages.searchFilter')),
4848
otherwise: Yup.string()
4949
}
@@ -52,7 +52,7 @@ export const getLDAPValidationSchema = (t: TFunction) => {
5252
['ldap_group_use_query', 'ldap_group_use_role_mapping'],
5353
{
5454
is: (ldap_group_use_query, ldap_group_use_role_mapping) =>
55-
ldap_group_use_role_mapping === true && ldap_group_use_query === 'false',
55+
ldap_group_use_role_mapping === true && ldap_group_use_query === 'true',
5656
then: Yup.string().required(t('messages.groupSearchBN')),
5757
otherwise: Yup.string()
5858
}

0 commit comments

Comments
 (0)