Fixed node name not changing on Share modal

This commit is contained in:
Lucas Oliveira 2023-12-09 22:10:27 -03:00
commit 14efa160f1
3 changed files with 17 additions and 14 deletions

View file

@ -174,6 +174,7 @@ export default function GenericNode({
if (nodeName.trim() !== "") { if (nodeName.trim() !== "") {
setNodeName(nodeName); setNodeName(nodeName);
data.node!.display_name = nodeName; data.node!.display_name = nodeName;
updateNodeInternals(data.id);
} else { } else {
setNodeName(data.node!.display_name); setNodeName(data.node!.display_name);
} }
@ -376,12 +377,9 @@ export default function GenericNode({
autoFocus autoFocus
onBlur={() => { onBlur={() => {
setInputDescription(false); setInputDescription(false);
if (nodeDescription.trim() !== "") { setNodeDescription(nodeDescription);
setNodeDescription(nodeDescription); data.node!.description = nodeDescription;
data.node!.description = nodeDescription; updateNodeInternals(data.id);
} else {
setNodeDescription(data.node!.description);
}
}} }}
value={nodeDescription} value={nodeDescription}
onChange={(e) => setNodeDescription(e.target.value)} onChange={(e) => setNodeDescription(e.target.value)}
@ -396,6 +394,7 @@ export default function GenericNode({
setInputDescription(false); setInputDescription(false);
setNodeDescription(nodeDescription); setNodeDescription(nodeDescription);
data.node!.description = nodeDescription; data.node!.description = nodeDescription;
updateNodeInternals(data.id);
} }
}} }}
/> />

View file

@ -44,8 +44,6 @@ export default function ShareModal({
const { setSuccessData, setErrorData } = useContext(alertContext); const { setSuccessData, setErrorData } = useContext(alertContext);
const { reactFlowInstance } = useContext(typesContext); const { reactFlowInstance } = useContext(typesContext);
const [checked, setChecked] = useState(false); const [checked, setChecked] = useState(false);
const [name, setName] = useState(component?.name ?? "");
const [description, setDescription] = useState(component?.description ?? "");
const [internalOpen, internalSetOpen] = useState(children ? false : true); const [internalOpen, internalSetOpen] = useState(children ? false : true);
const [openConfirmationModal, setOpenConfirmationModal] = useState(false); const [openConfirmationModal, setOpenConfirmationModal] = useState(false);
const nameComponent = is_component ? "component" : "flow"; const nameComponent = is_component ? "component" : "flow";
@ -61,6 +59,9 @@ export default function ShareModal({
const [loadingNames, setLoadingNames] = useState(false); const [loadingNames, setLoadingNames] = useState(false);
const name = component?.name ?? "";
const description = component?.description ?? "";
useEffect(() => { useEffect(() => {
if (open || internalOpen) { if (open || internalOpen) {
if (hasApiKey && hasStore) { if (hasApiKey && hasStore) {
@ -94,11 +95,6 @@ export default function ShareModal({
}); });
} }
useEffect(() => {
setName(component?.name ?? "");
setDescription(component?.description ?? "");
}, [component, open, internalOpen]);
const handleShareComponent = async (update = false) => { const handleShareComponent = async (update = false) => {
//remove file names from flows before sharing //remove file names from flows before sharing
removeFileNameFromComponents(component); removeFileNameFromComponents(component);

View file

@ -77,7 +77,15 @@ export default function NodeToolbarComponent({
useEffect(() => { useEffect(() => {
setFlowComponent(createFlowComponent(cloneDeep(data), version)); setFlowComponent(createFlowComponent(cloneDeep(data), version));
}, [data, data.node, showModalAdvanced]); }, [
data,
data.node,
data.node?.display_name,
data.node?.description,
data.node?.template,
showModalAdvanced,
showconfirmShare,
]);
const handleSelectChange = (event) => { const handleSelectChange = (event) => {
switch (event) { switch (event) {