chunzhi/comfyui-deploy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: invalid input for custom nodes

Browse Source
This commit is contained in:
bennykok 2024-01-28 17:48:22 +08:00
parent c1fc06fd39
commit 10bbb393a7
1 changed files with 39 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
web/src/components/custom-form/SnapshotPickerView.tsx
View File

@ -65,7 +65,8 @@ function SnapshotPresetPicker({
const [open, setOpen] = React.useState(false); const [open, setOpen] = React.useState(false);
const [selected, setSelected] = React.useState<string | null>(null); const [selected, setSelected] = React.useState<string | null>(null);
const [frameworks, setFramework] = React.useState< const [frameworks, setFramework] =
React.useState<
{ {
id: string; id: string;
label: string; label: string;
@ -140,7 +141,7 @@ function SnapshotPresetPicker({
"ml-auto h-4 w-4", "ml-auto h-4 w-4",
field.value === framework.value field.value === framework.value
? "opacity-100" ? "opacity-100"
: "opacity-0" : "opacity-0",
)} )}
/> />
</CommandItem> </CommandItem>
@ -199,12 +200,12 @@ function CustomNodesSelector({
const { data, error, isLoading } = useSWR<CustomNodeList>( const { data, error, isLoading } = useSWR<CustomNodeList>(
"https://raw.githubusercontent.com/ltdrdata/ComfyUI-Manager/main/custom-node-list.json", "https://raw.githubusercontent.com/ltdrdata/ComfyUI-Manager/main/custom-node-list.json",
fetcher fetcher,
); );
const keys = React.useMemo( const keys = React.useMemo(
() => Object.keys(customNodeList), () => Object.keys(customNodeList),
[customNodeList, data] [customNodeList, data],
); );
function findItem(value: string) { function findItem(value: string) {
@ -213,6 +214,11 @@ function CustomNodesSelector({
return included; return included;
} }
const onChangeRef = React.useRef(field.onChange);
React.useEffect(() => {
onChangeRef.current = field.onChange;
}, [field.onChange]);
return ( return (
<Popover open={open} onOpenChange={setOpen}> <Popover open={open} onOpenChange={setOpen}>
<PopoverTrigger asChild> <PopoverTrigger asChild>
@ -243,7 +249,7 @@ function CustomNodesSelector({
{ {
hash: string; hash: string;
disabled: boolean; disabled: boolean;
pip?: string[] pip?: string[];
} }
>; >;
const x = customNodeList; const x = customNodeList;
@ -256,7 +262,7 @@ function CustomNodesSelector({
const repoName = extractRepoName(currentValue); const repoName = extractRepoName(currentValue);
const id = toast.loading(`Fetching repo info...`); const id = toast.loading(`Fetching repo info...`);
const repo = await fetch( const repo = await fetch(
`https://api.github.com/repos/${repoName}` `https://api.github.com/repos/${repoName}`,
) )
.then((x) => x.json()) .then((x) => x.json())
.then((x) => { .then((x) => {
@ -267,14 +273,16 @@ function CustomNodesSelector({
.catch((e) => { .catch((e) => {
console.error(e); console.error(e);
toast.dismiss(id); toast.dismiss(id);
toast.error(`Failed to fetch repo info ${e.message}`); toast.error(
`Failed to fetch repo info ${e.message}`,
);
return null; return null;
}); });
if (!repo) return; if (!repo) return;
const branch = repo.default_branch; const branch = repo.default_branch;
const branchInfo = await fetch( const branchInfo = await fetch(
`https://api.github.com/repos/${repoName}/branches/${branch}` `https://api.github.com/repos/${repoName}/branches/${branch}`,
) )
.then((x) => x.json()) .then((x) => x.json())
.then((x) => BranchInfoSchema.parse(x)) .then((x) => BranchInfoSchema.parse(x))
@ -282,7 +290,7 @@ function CustomNodesSelector({
console.error(e); console.error(e);
toast.dismiss(id); toast.dismiss(id);
toast.error( toast.error(
`Failed to fetch branch info ${e.message}` `Failed to fetch branch info ${e.message}`,
); );
return null; return null;
}); });
@ -291,19 +299,27 @@ function CustomNodesSelector({
if (!branchInfo) return; if (!branchInfo) return;
nodeList = { const value: Record<string, any> = {
[currentValue]: {
hash: branchInfo?.commit.sha, hash: branchInfo?.commit.sha,
disabled: false, disabled: false,
pip: framework.pip };
},
if (framework.pip) {
value["pip"] = framework.pip;
}
nodeList = {
[currentValue]: value,
...x, ...x,
}; };
} }
field.onChange({
const newValue = {
...field.value, ...field.value,
git_custom_nodes: nodeList, git_custom_nodes: nodeList,
}); };
field.onChange(newValue);
}} }}
> >
{framework.title} {framework.title}
@ -312,7 +328,7 @@ function CustomNodesSelector({
"ml-auto h-4 w-4", "ml-auto h-4 w-4",
findItem(framework.reference) findItem(framework.reference)
? "opacity-100" ? "opacity-100"
: "opacity-0" : "opacity-0",
)} )}
/> />
</CommandItem> </CommandItem>