From 399bd42124ac68440e0422bd99684803f930998f Mon Sep 17 00:00:00 2001
From: Erik Jan de Wit <erikjan.dewit@gmail.com>
Date: Fri, 21 Apr 2023 13:15:42 +0200
Subject: [PATCH] Disable `stringify` by default and enable it for
 authorization (#19871)

---
 .../src/authentication/components/ExecutionConfigModal.tsx     | 1 +
 js/apps/admin-ui/src/components/dynamic/DynamicComponents.tsx  | 3 +--
 .../src/components/dynamic/MultivaluedStringComponent.tsx      | 3 ++-
 js/apps/admin-ui/src/components/dynamic/components.ts          | 1 +
 4 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/js/apps/admin-ui/src/authentication/components/ExecutionConfigModal.tsx b/js/apps/admin-ui/src/authentication/components/ExecutionConfigModal.tsx
index 104ca45528..9e56176d0d 100644
--- a/js/apps/admin-ui/src/authentication/components/ExecutionConfigModal.tsx
+++ b/js/apps/admin-ui/src/authentication/components/ExecutionConfigModal.tsx
@@ -160,6 +160,7 @@ export const ExecutionConfigModal = ({
             </FormGroup>
             <FormProvider {...form}>
               <DynamicComponents
+                stringify
                 properties={configDescription.properties || []}
               />
             </FormProvider>
diff --git a/js/apps/admin-ui/src/components/dynamic/DynamicComponents.tsx b/js/apps/admin-ui/src/components/dynamic/DynamicComponents.tsx
index 86f076088e..0ac4336ff5 100644
--- a/js/apps/admin-ui/src/components/dynamic/DynamicComponents.tsx
+++ b/js/apps/admin-ui/src/components/dynamic/DynamicComponents.tsx
@@ -5,8 +5,7 @@ import { convertAttributeNameToForm } from "../../util";
 
 type DynamicComponentProps = {
   properties: ConfigPropertyRepresentation[];
-  selectedValues?: string[];
-  parentCallback?: (data: string[]) => void;
+  stringify?: boolean;
 };
 
 export const DynamicComponents = ({
diff --git a/js/apps/admin-ui/src/components/dynamic/MultivaluedStringComponent.tsx b/js/apps/admin-ui/src/components/dynamic/MultivaluedStringComponent.tsx
index 3bbe77e65a..94ae7bbbf1 100644
--- a/js/apps/admin-ui/src/components/dynamic/MultivaluedStringComponent.tsx
+++ b/js/apps/admin-ui/src/components/dynamic/MultivaluedStringComponent.tsx
@@ -11,6 +11,7 @@ export const MultiValuedStringComponent = ({
   label,
   defaultValue,
   helpText,
+  stringify,
   isDisabled = false,
 }: ComponentProps) => {
   const { t } = useTranslation("dynamic");
@@ -32,7 +33,7 @@ export const MultiValuedStringComponent = ({
         addButtonLabel={t("addMultivaluedLabel", {
           fieldLabel: t(label!).toLowerCase(),
         })}
-        stringify
+        stringify={stringify}
       />
     </FormGroup>
   );
diff --git a/js/apps/admin-ui/src/components/dynamic/components.ts b/js/apps/admin-ui/src/components/dynamic/components.ts
index 18fac698c4..6274ba8178 100644
--- a/js/apps/admin-ui/src/components/dynamic/components.ts
+++ b/js/apps/admin-ui/src/components/dynamic/components.ts
@@ -16,6 +16,7 @@ import { TextComponent } from "./TextComponent";
 
 export type ComponentProps = Omit<ConfigPropertyRepresentation, "type"> & {
   isDisabled?: boolean;
+  stringify?: boolean;
 };
 
 const ComponentTypes = [