From ed4bd561469449de056bc1e579c7b2d1b7b610ea Mon Sep 17 00:00:00 2001 From: twwu Date: Tue, 19 Aug 2025 14:41:32 +0800 Subject: [PATCH] refactor(create-card, template-card): add TODO comments for direct pipeline dataset creation and improve code organization --- .../create-from-pipeline/list/create-card.tsx | 31 +++++++++++++++++++ .../list/template-card/index.tsx | 1 + 2 files changed, 32 insertions(+) diff --git a/web/app/components/datasets/create-from-pipeline/list/create-card.tsx b/web/app/components/datasets/create-from-pipeline/list/create-card.tsx index 7599ff7d65..9302a10e7b 100644 --- a/web/app/components/datasets/create-from-pipeline/list/create-card.tsx +++ b/web/app/components/datasets/create-from-pipeline/list/create-card.tsx @@ -2,12 +2,43 @@ import React, { useCallback, useState } from 'react' import { useTranslation } from 'react-i18next' import { RiAddCircleLine } from '@remixicon/react' import CreateFromScratchModal from '../create-options/create-from-scratch-modal' +// import { useCreatePipelineDataset } from '@/service/knowledge/use-create-dataset' +// import { useResetDatasetList } from '@/service/knowledge/use-dataset' +// import Toast from '@/app/components/base/toast' +// import { useRouter } from 'next/navigation' const CreateCard = () => { const { t } = useTranslation() + // const { push } = useRouter() const [showCreateModal, setShowCreateModal] = useState(false) + // const { mutateAsync: createEmptyDataset } = useCreatePipelineDataset() + // const resetDatasetList = useResetDatasetList() + + // todo: Directly create a pipeline dataset, no need to fill in the form + // const handleCreate = useCallback(async () => { + // await createEmptyDataset({}, { + // onSuccess: (data) => { + // if (data) { + // const { id } = data + // Toast.notify({ + // type: 'success', + // message: t('datasetPipeline.creation.successTip'), + // }) + // resetDatasetList() + // push(`/datasets/${id}/pipeline`) + // } + // }, + // onError: () => { + // Toast.notify({ + // type: 'error', + // message: t('datasetPipeline.creation.errorTip'), + // }) + // }, + // }) + // }, [createEmptyDataset, push, resetDatasetList, t]) + const openCreateFromScratch = useCallback(() => { setShowCreateModal(true) }, []) diff --git a/web/app/components/datasets/create-from-pipeline/list/template-card/index.tsx b/web/app/components/datasets/create-from-pipeline/list/template-card/index.tsx index 1393eeb921..5bd0afa7a7 100644 --- a/web/app/components/datasets/create-from-pipeline/list/template-card/index.tsx +++ b/web/app/components/datasets/create-from-pipeline/list/template-card/index.tsx @@ -53,6 +53,7 @@ const TemplateCard = ({ setShowCreateModal(true) }, []) + // todo: Directly create a pipeline dataset, no need to fill in the form const handleUseTemplate = useCallback(async (payload: Omit) => { const { data: pipelineTemplateInfo } = await getPipelineTemplateInfo() if (!pipelineTemplateInfo) {