Skip to content

Commit 7103588

Browse files
committed
use CreateDomainSecurityGroup instead of UpdateDomainSecurityGroup API command for group creation, enable offline form handling
1 parent e488481 commit 7103588

3 files changed

Lines changed: 5 additions & 1 deletion

File tree

src/libs/API/types.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -573,6 +573,7 @@ const WRITE_COMMANDS = {
573573
EXPORT_DOMAIN_MEMBERS_CSV: 'ExportDomainMembersCSV',
574574
INITIATE_BANK_ACCOUNT_UNLOCK: 'InitiateBankAccountUnlock',
575575
UPDATE_DOMAIN_SECURITY_GROUP: 'UpdateDomainSecurityGroupForNewDot',
576+
CREATE_DOMAIN_SECURITY_GROUP: 'CreateDomainSecurityGroup',
576577
SET_DEFAULT_DOMAIN_SECURITY_GROUP: 'SetDefaultDomainSecurityGroup',
577578
} as const;
578579

@@ -1164,6 +1165,7 @@ type WriteCommandParameters = {
11641165
[WRITE_COMMANDS.RESET_DOMAIN_MEMBER_TWO_FACTOR_AUTH]: Parameters.ResetDomainMemberTwoFactorAuthParams;
11651166
[WRITE_COMMANDS.EXPORT_DOMAIN_MEMBERS_CSV]: Parameters.ExportDomainMembersCSVParams;
11661167
[WRITE_COMMANDS.UPDATE_DOMAIN_SECURITY_GROUP]: Parameters.UpdateDomainSecurityGroupParams;
1168+
[WRITE_COMMANDS.CREATE_DOMAIN_SECURITY_GROUP]: Parameters.CreateDomainSecurityGroupParams;
11671169
[WRITE_COMMANDS.SET_DEFAULT_DOMAIN_SECURITY_GROUP]: Parameters.SetDefaultDomainSecurityGroupParams;
11681170
};
11691171

src/libs/actions/Domain.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1915,11 +1915,12 @@ function createDomainSecurityGroup(domainAccountID: number, newSecurityGroup: Do
19151915
},
19161916
];
19171917

1918+
// using the same interface as UpdateDomainSecurityGroup to correctly handle errors
19181919
const params: UpdateDomainSecurityGroupParams = {
19191920
domainAccountID,
19201921
name: SECURITY_GROUP_KEY,
19211922
value: JSON.stringify(newSecurityGroup),
1922-
settingsName: 'createGroup', // placeholder, waiting for BE
1923+
settingsName: 'createGroup',
19231924
};
19241925

19251926
API.write(WRITE_COMMANDS.UPDATE_DOMAIN_SECURITY_GROUP, params, {optimisticData, failureData, successData});

src/pages/domain/Groups/DomainGroupCreatePage.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ function DomainGroupCreatePage({route}: DomainGroupCreatePageProps) {
111111
clearDomainGroupCreatePreferredPolicyID();
112112
Navigation.goBack(ROUTES.DOMAIN_GROUPS.getRoute(domainAccountID));
113113
}}
114+
enabledWhenOffline
114115
submitButtonText={translate('domain.groups.createGroupSubmitButton')}
115116
style={[styles.flex1]}
116117
submitButtonStyles={[styles.ph5, styles.pb3]}

0 commit comments

Comments
 (0)