Skip to content

Commit 6fcb9e6

Browse files
committed
refactor: remove unnecessary showErrors & ?
1 parent 11a2e6f commit 6fcb9e6

File tree

8 files changed

+12
-26
lines changed

8 files changed

+12
-26
lines changed

src/components/ApplicationGroup/Details/TriggerView/EnvTriggerView.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -216,9 +216,7 @@ export default function EnvTriggerView({ filteredAppIds, isVirtualEnv }: AppGrou
216216
setDefaultConfig(_isDefaultConfig)
217217
setConfigPresent(isConfigPresent)
218218
})
219-
.catch((error) => {
220-
showError(error)
221-
})
219+
.catch()
222220
}
223221

224222
const preserveSelection = (_workflows: WorkflowType[]) => {

src/components/EnvironmentOverride/DeploymentTemplateOverrideForm.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -230,8 +230,8 @@ export default function DeploymentTemplateOverrideForm({
230230
: await api(+appId, +envId, payload)
231231
if (deploymentTemplateResp.result?.isLockConfigError && !saveEligibleChanges) {
232232
// checking if any locked changes and opening drawer to show eligible and locked ones
233-
setLockedOverride(deploymentTemplateResp.result?.lockedOverride)
234-
setDisableSaveEligibleChanges(deploymentTemplateResp.result?.disableSaveEligibleChanges)
233+
setLockedOverride(deploymentTemplateResp.result.lockedOverride)
234+
setDisableSaveEligibleChanges(deploymentTemplateResp.result.disableSaveEligibleChanges)
235235
handleLockedDiffDrawer(true)
236236
return
237237
}
@@ -242,10 +242,10 @@ export default function DeploymentTemplateOverrideForm({
242242

243243
if (envOverrideValuesWithBasic) {
244244
editorOnChange(YAML.stringify(envOverrideValuesWithBasic, { indent: 2 }), true)
245-
} else {
245+
} else if (deploymentTemplateResp.result?.envOverrideValues) {
246246
dispatch({
247247
type: DeploymentConfigStateActionTypes.tempFormData,
248-
payload: YAML.stringify(deploymentTemplateResp.result?.envOverrideValues),
248+
payload: YAML.stringify(deploymentTemplateResp.result.envOverrideValues),
249249
})
250250
}
251251
toast.success(

src/components/app/details/appDetails/AppDetails.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -333,9 +333,7 @@ export const Details: React.FC<DetailsType> = ({
333333
}
334334
},
335335
)
336-
.catch((error) => {
337-
showError(error)
338-
})
336+
.catch()
339337
},
340338
[
341339
params.appId,

src/components/app/details/main.tsx

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -109,9 +109,7 @@ export default function AppDetailsPage({ isV2 }: AppDetailsProps) {
109109
_groupFilterOption.sort(sortOptionsByLabel)
110110
setGroupFilterOptions(_groupFilterOption)
111111
}
112-
} catch (error) {
113-
showError(error)
114-
}
112+
} catch {}
115113
setAppListLoading(false)
116114
}
117115

@@ -132,9 +130,7 @@ export default function AppDetailsPage({ isV2 }: AppDetailsProps) {
132130
.sort(sortOptionsByLabel),
133131
)
134132
}
135-
} catch (error) {
136-
showError(error)
137-
}
133+
} catch {}
138134
setAppListLoading(false)
139135
}
140136

src/components/app/details/triggerView/TriggerView.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -171,9 +171,7 @@ class TriggerView extends Component<TriggerViewProps, TriggerViewState> {
171171
const _isDefaultConfig = response.result.is_default_configured
172172
this.setState({ configs: isConfigPresent, isDefaultConfigPresent: _isDefaultConfig })
173173
})
174-
.catch((error) => {
175-
showError(error)
176-
})
174+
.catch()
177175
}
178176

179177
getWorkflows = (isFromOnMount?: boolean) => {

src/components/deploymentConfig/DeploymentTemplateView/DeploymentConfigFormCTA.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ export default function DeploymentConfigFormCTA({
7474
const deploymentTemplateResp = await checkForProtectedLockedChanges()
7575
if (deploymentTemplateResp.result?.isLockConfigError) {
7676
setShowLockedDiffForApproval(true)
77-
setLockedOverride(deploymentTemplateResp.result?.lockedOverride)
77+
setLockedOverride(deploymentTemplateResp.result.lockedOverride)
7878
handleLockedDiffDrawer(true)
7979
} else {
8080
setShowLockedDiffForApproval(false)

src/components/login/Login.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,7 @@ export default class Login extends Component<LoginProps, LoginFormState> {
6161
loginList: list,
6262
})
6363
})
64-
.catch(error => {
65-
showError(error)
66-
})
64+
.catch()
6765
if (typeof Storage !== 'undefined') {
6866
if (localStorage.isDashboardAccessed) {
6967
return

src/components/v2/appDetails/AppDetails.component.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,9 +105,7 @@ const AppDetailsComponent = ({
105105
}
106106
},
107107
)
108-
.catch((error) => {
109-
showError(error)
110-
})
108+
.catch()
111109
}
112110

113111
const processDeploymentStatusData = (deploymentStatusDetailRes: DeploymentStatusDetailsType): void => {

0 commit comments

Comments
 (0)