Skip to content

Commit 53e2b2a

Browse files
chore: improve code qty
1 parent 2915791 commit 53e2b2a

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

src/components/ApplicationGroup/AppGroup.service.ts

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -195,8 +195,11 @@ export const getAppGroupList = (envId: number): Promise<AppGroupList> => {
195195
}
196196

197197
export const getEnvGroupList = (envId: number, filterParentType?:string): Promise<EnvGroupListResponse> => {
198-
if(!filterParentType) return get(`${Routes.ENVIRONMENT}/${envId}/${Routes.GROUPS}`)
199-
return get(`${Routes.ENVIRONMENT}/${envId}/${Routes.GROUPS}?groupType=${filterParentType}`)
198+
let filterParentTypeQuery = ''
199+
if (filterParentType) {
200+
filterParentTypeQuery = `?groupType=${filterParentType}`
201+
}
202+
return get(`${Routes.ENVIRONMENT}/${envId}/${Routes.GROUPS}${filterParentTypeQuery}`)
200203
}
201204

202205
export const getEnvGroup = (envId: number, groupId: number): Promise<EnvGroupResponse> => {
@@ -215,5 +218,9 @@ export const appGroupPermission = (envId: string, data: CheckPermissionType): Pr
215218
}
216219

217220
export const deleteEnvGroup = (envId: string, groupId: string, filterParentType?:string): Promise<EnvGroupResponse> => {
218-
return trash(`${Routes.ENVIRONMENT}/${envId}/${Routes.GROUP}/${groupId}?groupType=${filterParentType}`)
221+
let filterParentTypeQuery = ''
222+
if (filterParentType) {
223+
filterParentTypeQuery = `?groupType=${filterParentType}`
224+
}
225+
return trash(`${Routes.ENVIRONMENT}/${envId}/${Routes.GROUP}/${groupId}${filterParentTypeQuery}`)
219226
}

src/components/ApplicationGroup/AppGroupDetailsRoute.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ export default function AppGroupDetailsRoute({ isSuperAdmin }: AppGroupAdminType
393393
appList={allAppsList}
394394
selectedAppGroup={clickedGroup}
395395
closePopup={closeCreateGroup}
396-
filterParentType={FilterParentType.app}
396+
filterParentType={FilterParentType.env}
397397
/>
398398
)}
399399
{showDeleteGroup && isPopupBox && (

0 commit comments

Comments
 (0)