📝 (frontend): reorganize imports and fix import order in OutputComponent and ParameterComponent to improve code readability

♻️ (frontend): refactor OutputComponent and ParameterComponent to remove unnecessary imports and optimize code structure
📝 (frontend): remove console.log statement in ApiModal view to clean up code and improve maintainability
This commit is contained in:
ogabrielluiz 2024-06-04 12:55:39 -03:00
commit 6e7421998d
4 changed files with 22 additions and 18 deletions

View file

@ -1,16 +1,16 @@
import ForwardedIconComponent from "../../../../components/genericIconComponent";
import { outputComponentType } from "../../../../types/components";
import { cn } from "../../../../utils/utils";
import useFlowStore from "../../../../stores/flowStore";
import { NodeDataType } from "../../../../types/flow";
import { cloneDeep } from "lodash"; import { cloneDeep } from "lodash";
import { useUpdateNodeInternals } from "reactflow";
import ForwardedIconComponent from "../../../../components/genericIconComponent";
import { import {
DropdownMenu, DropdownMenu,
DropdownMenuContent, DropdownMenuContent,
DropdownMenuItem, DropdownMenuItem,
DropdownMenuTrigger, DropdownMenuTrigger,
} from "../../../../components/ui/dropdown-menu"; } from "../../../../components/ui/dropdown-menu";
import { useUpdateNodeInternals } from "reactflow"; import useFlowStore from "../../../../stores/flowStore";
import { outputComponentType } from "../../../../types/components";
import { NodeDataType } from "../../../../types/flow";
import { cn } from "../../../../utils/utils";
export default function OutputComponent({ export default function OutputComponent({
selected, selected,
@ -28,7 +28,7 @@ export default function OutputComponent({
} }
return ( return (
<div className="flex gap-2"> <div className="nocopy nopan nodelete nodrag noundo flex gap-2 ">
<span>{name}</span> <span>{name}</span>
<DropdownMenu> <DropdownMenu>
<DropdownMenuTrigger> <DropdownMenuTrigger>

View file

@ -80,7 +80,7 @@ export default function ParameterComponent({
debouncedHandleUpdateValues, debouncedHandleUpdateValues,
setNode, setNode,
isLoading, isLoading,
setIsLoading, setIsLoading
); );
const { handleNodeClass: handleNodeClassHook } = useHandleNodeClass( const { handleNodeClass: handleNodeClassHook } = useHandleNodeClass(
@ -88,7 +88,7 @@ export default function ParameterComponent({
name, name,
takeSnapshot, takeSnapshot,
setNode, setNode,
updateNodeInternals, updateNodeInternals
); );
const { handleRefreshButtonPress: handleRefreshButtonPressHook } = const { handleRefreshButtonPress: handleRefreshButtonPressHook } =
@ -97,7 +97,7 @@ export default function ParameterComponent({
let disabled = let disabled =
edges.some( edges.some(
(edge) => (edge) =>
edge.targetHandle === scapedJSONStringfy(proxy ? { ...id, proxy } : id), edge.targetHandle === scapedJSONStringfy(proxy ? { ...id, proxy } : id)
) ?? false; ) ?? false;
const handleRefreshButtonPress = async (name, data) => { const handleRefreshButtonPress = async (name, data) => {
@ -108,7 +108,7 @@ export default function ParameterComponent({
const handleOnNewValue = async ( const handleOnNewValue = async (
newValue: string | string[] | boolean | Object[], newValue: string | string[] | boolean | Object[],
skipSnapshot: boolean | undefined = false, skipSnapshot: boolean | undefined = false
): Promise<void> => { ): Promise<void> => {
handleOnNewValueHook(newValue, skipSnapshot); handleOnNewValueHook(newValue, skipSnapshot);
}; };
@ -135,7 +135,11 @@ export default function ParameterComponent({
<OutputComponent <OutputComponent
idx={index} idx={index}
types={type?.split("|") ?? []} types={type?.split("|") ?? []}
selected={title} selected={
data.node?.outputs![index].selected ??
data.node?.outputs![index].types[0] ??
title
}
nodeId={data.id} nodeId={data.id}
frozen={data.node?.frozen} frozen={data.node?.frozen}
name={outputName ?? type ?? title} name={outputName ?? type ?? title}
@ -190,14 +194,14 @@ export default function ParameterComponent({
className={classNames( className={classNames(
left ? "my-12 -ml-0.5 " : " my-12 -mr-0.5 ", left ? "my-12 -ml-0.5 " : " my-12 -mr-0.5 ",
"h-3 w-3 rounded-full border-2 bg-background", "h-3 w-3 rounded-full border-2 bg-background",
!showNode ? "mt-0" : "", !showNode ? "mt-0" : ""
)} )}
style={{ style={{
borderColor: color ?? nodeColors.unknown, borderColor: color ?? nodeColors.unknown,
}} }}
onClick={() => { onClick={() => {
setFilterEdge( setFilterEdge(
groupByFamily(myData, tooltipTitle!, left, nodes!), groupByFamily(myData, tooltipTitle!, left, nodes!)
); );
}} }}
></Handle> ></Handle>
@ -282,12 +286,12 @@ export default function ParameterComponent({
} }
className={classNames( className={classNames(
left ? "-ml-0.5" : "-mr-0.5", left ? "-ml-0.5" : "-mr-0.5",
"h-3 w-3 rounded-full border-2 bg-background", "h-3 w-3 rounded-full border-2 bg-background"
)} )}
style={{ borderColor: color ?? nodeColors.unknown }} style={{ borderColor: color ?? nodeColors.unknown }}
onClick={() => { onClick={() => {
setFilterEdge( setFilterEdge(
groupByFamily(myData, tooltipTitle!, left, nodes!), groupByFamily(myData, tooltipTitle!, left, nodes!)
); );
}} }}
/> />

View file

@ -831,7 +831,8 @@ export default function GenericNode({
<ParameterComponent <ParameterComponent
index={idx} index={idx}
key={scapedJSONStringfy({ key={scapedJSONStringfy({
output_types: [output.selected ?? output.types[0]], output_types: output.types,
name: output.name,
id: data.id, id: data.id,
dataType: data.type, dataType: data.type,
})} })}

View file

@ -60,7 +60,6 @@ const ApiModal = forwardRef(
); );
const pythonCode = getPythonCode(flow?.name, tweak); const pythonCode = getPythonCode(flow?.name, tweak);
const widgetCode = getWidgetCode(flow?.id, flow?.name, autoLogin); const widgetCode = getWidgetCode(flow?.id, flow?.name, autoLogin);
console.log("flow", flow);
const includeWebhook = flow.webhook; const includeWebhook = flow.webhook;
const tweaksCode = buildTweaks(flow); const tweaksCode = buildTweaks(flow);
const codesArray = [ const codesArray = [