Skip to content

Commit

Permalink
Merge pull request #5767 from dasuni-30/fix/22700
Browse files Browse the repository at this point in the history
Fix custom properties update issue for connections
  • Loading branch information
dasuni-30 authored Mar 17, 2024
2 parents b924f32 + b9f5086 commit 727bf84
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 10 deletions.
5 changes: 5 additions & 0 deletions .changeset/tricky-pens-applaud.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@wso2is/console": patch
---

Fix custom properties update issue in connections
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ export const CommonPluggableComponentForm: FunctionComponent<CommonPluggableComp
});

const modifiedCustomProperties: any = !isEmpty(resolvedCustomProperties) ?
resolvedCustomProperties?.toString()?.split(",")?.map(
resolvedCustomProperties?.toString()?.split(", ")?.map(
(customProperty: string) => {
const keyValuePair: string[] = customProperty.split("=");

Expand Down Expand Up @@ -467,12 +467,6 @@ export const CommonPluggableComponentForm: FunctionComponent<CommonPluggableComp
return;
}

// Check whether the property already in dynamicValues?.properties.
if (dynamicValues?.properties?.find(
(prop: CommonPluggableComponentPropertyInterface) => prop.key === property.key)) {
return;
}

// Check whether the property is not in the metadata.
if (!metadata?.properties?.find(
(meta: CommonPluggableComponentMetaPropertyInterface) => meta.key === property.key)) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1000,9 +1000,6 @@ export const AuthenticatorSettings: FunctionComponent<IdentityProviderSettingsPr
mode={ AuthenticatorSettingsFormModes.EDIT }
authenticator={ authenticator }
metadata={ authenticator.meta }
showCustomProperties={
authenticator.id !== ConnectionManagementConstants.GITHUB_AUTHENTICATOR_ID
}
initialValues={ authenticator.data }
onSubmit={ handleAuthenticatorConfigFormSubmit }
type={ authenticator.meta?.authenticatorId }
Expand Down

0 comments on commit 727bf84

Please sign in to comment.