From 38b7d0f600947f45213e4971ff4f3b669d320b9d Mon Sep 17 00:00:00 2001 From: Hornwitser Date: Wed, 24 Feb 2021 20:18:41 +0100 Subject: [PATCH] Fix web config trying to set the new prop value as field Fix the BaseConfigTree treating the nex property name and value boxes as fields to be applied to the config when pressing Apply. --- packages/web_ui/src/components/BaseConfigTree.jsx | 3 +++ 1 file changed, 3 insertions(+) diff --git a/packages/web_ui/src/components/BaseConfigTree.jsx b/packages/web_ui/src/components/BaseConfigTree.jsx index 7fcc1548..7f6fd457 100644 --- a/packages/web_ui/src/components/BaseConfigTree.jsx +++ b/packages/web_ui/src/components/BaseConfigTree.jsx @@ -42,6 +42,9 @@ export default function BaseConfigTree(props) { let newChangedFields = new Set(changedFields); let changed = false; for (let [key, value] of Object.entries(changedValues)) { + if (key.endsWith(":add.name") || key.endsWith(":add.value")) { + continue; + } if (initialValues[key] === value) { changed = newChangedFields.delete(key) || changed; } else {