Skip to content

Commit 84d20b0

Browse files
committed
reset settings when navigate to choose-settings-step page
1 parent f4b1b5b commit 84d20b0

File tree

1 file changed

+4
-12
lines changed
  • x-pack/plugins/apm/public/components/app/Settings/AgentConfigurations/AgentConfigurationCreateEdit

1 file changed

+4
-12
lines changed

x-pack/plugins/apm/public/components/app/Settings/AgentConfigurations/AgentConfigurationCreateEdit/index.tsx

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -86,17 +86,6 @@ export function AgentConfigurationCreateEdit({
8686
setNewConfig(getInitialNewConfig(existingConfig));
8787
}, [existingConfig]);
8888

89-
useEffect(
90-
() => {
91-
// cleanup settings when navigating to "choose-settings-step" to not save invalid configurations
92-
if (pageStep === 'choose-settings-step' && !isEditMode) {
93-
setNewConfig({ ...newConfig, settings: {} });
94-
}
95-
},
96-
// eslint-disable-next-line react-hooks/exhaustive-deps
97-
[pageStep]
98-
);
99-
10089
useEffect(() => {
10190
// the user tried to edit the service of an existing config
10291
if (pageStep === 'choose-service-step' && isEditMode) {
@@ -143,7 +132,10 @@ export function AgentConfigurationCreateEdit({
143132
<ServicePage
144133
newConfig={newConfig}
145134
setNewConfig={setNewConfig}
146-
onClickNext={() => setPage('choose-settings-step')}
135+
onClickNext={() => {
136+
resetSettings();
137+
setPage('choose-settings-step');
138+
}}
147139
/>
148140
)}
149141

0 commit comments

Comments
 (0)