Skip to content

Commit d86c811

Browse files
committed
code refactoring
1 parent e15c6c1 commit d86c811

File tree

3 files changed

+2
-3
lines changed

3 files changed

+2
-3
lines changed

src/components/charts/AdvancedConfig.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ const AdvancedConfig: React.FC<AdvancedConfig> = ({ chart, index, fetchChartValu
152152
data-testid="advanced-option-app-name-box"
153153
isRequiredField={true}
154154
error={appName?.error}
155-
additionalErrorInfo={appName?.error && renderAdditionalErrorInfo(handleNameChange, appName.suggestedName, index)}
155+
additionalErrorInfo={renderAdditionalErrorInfo(handleNameChange, appName.suggestedName, index)}
156156
/>
157157
</div>}
158158
{handleEnvironmentChange && <div className="flex top mb-16">

src/components/charts/charts.util.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,6 @@ export const renderAdditionalErrorInfo = (handleNameChange, suggestedName, index
108108
. Suggested Name:
109109
<span className="anchor pointer" onClick={(e) => handleNameChange(index, suggestedName)}>
110110
{suggestedName}
111-
{console.log(suggestedName)}
112111
</span>
113112
</>
114113
)

src/components/charts/modal/ChartGroupBasicDeploy.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,7 @@ function ApplicationNameList({ charts, handleNameChange, showAppNames }) {
222222
}}
223223
error={chart.name.error}
224224
isRequiredField={true}
225-
additionalErrorInfo={chart.name.error ? chart.name.error : 'no'}
225+
additionalErrorInfo={renderAdditionalErrorInfo(handleNameChange, chart.name.suggestedName, index)}
226226
/>
227227
</div>
228228
</div>

0 commit comments

Comments
 (0)