From a41032def705a5675aabbd5b37ab065b06e0ce29 Mon Sep 17 00:00:00 2001 From: anovazzi1 Date: Fri, 20 Oct 2023 12:35:36 -0300 Subject: [PATCH] fix(tabsContext.tsx): fix issue with missing return statement causing incorrect data processing fix(tabsContext.tsx): move setData function call outside of try-catch block to ensure it always executes fix(MainPage/index.tsx): filter out component flows from the list of flows to be rendered --- src/frontend/src/contexts/tabsContext.tsx | 24 ++++++----- src/frontend/src/pages/MainPage/index.tsx | 52 ++++++++++++----------- 2 files changed, 40 insertions(+), 36 deletions(-) diff --git a/src/frontend/src/contexts/tabsContext.tsx b/src/frontend/src/contexts/tabsContext.tsx index 4196f269c..1fc9e8ab9 100644 --- a/src/frontend/src/contexts/tabsContext.tsx +++ b/src/frontend/src/contexts/tabsContext.tsx @@ -146,19 +146,21 @@ export function TabsProvider({ children }: { children: ReactNode }) { return; } if (flow.data && flow.is_component) { - storeComponents[(flow.data.nodes[0].data as NodeDataType).type] = ( - flow.data.nodes[0].data as NodeDataType - ).node!; + storeComponents[(flow.data.nodes[0].data as NodeDataType).type] = + _.cloneDeep((flow.data.nodes[0].data as NodeDataType).node!); + return; } - setData((prev) => { - let newData = _.cloneDeep(prev); - Object.keys(storeComponents).forEach((key) => { - newData["custom_components"][key] = storeComponents[key]; - }); - return newData; - }); processDataFromFlow(flow, false); - } catch (e) {} + } catch (e) { + console.log(e); + } + }); + setData((prev) => { + let newData = _.cloneDeep(prev); + Object.keys(storeComponents).forEach((key) => { + newData["custom_components"][key] = storeComponents[key]; + }); + return newData; }); } diff --git a/src/frontend/src/pages/MainPage/index.tsx b/src/frontend/src/pages/MainPage/index.tsx index 7ee00f5c9..441363a27 100644 --- a/src/frontend/src/pages/MainPage/index.tsx +++ b/src/frontend/src/pages/MainPage/index.tsx @@ -145,31 +145,33 @@ export default function HomePage(): JSX.Element { ) : ( - flows.map((flow, idx) => ( - - - - } - onDelete={() => { - removeFlow(flow.id); - }} - /> - )) + flows + .filter((flow) => !flow.is_component) + .map((flow, idx) => ( + + + + } + onDelete={() => { + removeFlow(flow.id); + }} + /> + )) )} )}