fix(ui): Cannot add an app that has both name and server destination (#…

…21440) Signed-off-by: Rafal Pelczar <[email protected]>
argoproj · Jan 24, 2025 · 544aea1 · 544aea1
1 parent 75def4f
commit 544aea1
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/ui/src/app/applications/components/application-create-panel/application-create-panel.tsx b/ui/src/app/applications/components/application-create-panel/application-create-panel.tsx
@@ -170,6 +170,16 @@ export const ApplicationCreatePanel = (props: {
         }
     }
 
+    const onCreateApp = (data: models.Application) => {
+        if (destinationComboValue === 'URL') {
+            delete data.spec.destination.name;
+        } else {
+            delete data.spec.destination.server;
+        }
+
+        props.createApp(data);
+    };
+
     return (
         <React.Fragment>
             <DataLoader
@@ -223,7 +233,7 @@ export const ApplicationCreatePanel = (props: {
                                     })}
                                     defaultValues={app}
                                     formDidUpdate={state => debouncedOnAppChanged(state.values as any)}
-                                    onSubmit={props.createApp}
+                                    onSubmit={onCreateApp}
                                     getApi={props.getFormApi}>
                                     {api => {
                                         const generalPanel = () => (