From 958ff4470769aa30006a923742fd4f798e58d9cf Mon Sep 17 00:00:00 2001 From: twwu Date: Wed, 14 May 2025 16:27:59 +0800 Subject: [PATCH] refactor: simplify import DSL confirmation request structure --- .../create-options/create-from-dsl-modal/index.tsx | 4 +--- web/models/pipeline.ts | 7 ------- web/service/use-pipeline.ts | 7 +++---- 3 files changed, 4 insertions(+), 14 deletions(-) diff --git a/web/app/components/datasets/create-from-pipeline/create-options/create-from-dsl-modal/index.tsx b/web/app/components/datasets/create-from-pipeline/create-options/create-from-dsl-modal/index.tsx index 94d54542f9..fd713d9d4d 100644 --- a/web/app/components/datasets/create-from-pipeline/create-options/create-from-dsl-modal/index.tsx +++ b/web/app/components/datasets/create-from-pipeline/create-options/create-from-dsl-modal/index.tsx @@ -156,9 +156,7 @@ const CreateFromDSLModal = ({ try { if (!importId) return - const response = await importDSLConfirm({ - import_id: importId, - }) + const response = await importDSLConfirm(importId) const { status, pipeline_id, dataset_id } = response diff --git a/web/models/pipeline.ts b/web/models/pipeline.ts index dbb34367f5..83d073648b 100644 --- a/web/models/pipeline.ts +++ b/web/models/pipeline.ts @@ -80,16 +80,9 @@ export type ImportPipelineDSLResponse = { dataset_id: string current_dsl_version: string imported_dsl_version: string - error: string - leaked_dependencies: Dependency[] -} - -export type ImportPipelineDSLConfirmRequest = { - import_id: string } export type ImportPipelineDSLConfirmResponse = { - id: string status: DSLImportStatus pipeline_id: string dataset_id: string diff --git a/web/service/use-pipeline.ts b/web/service/use-pipeline.ts index edbfc8d284..685e7c6fc3 100644 --- a/web/service/use-pipeline.ts +++ b/web/service/use-pipeline.ts @@ -5,7 +5,6 @@ import type { DeletePipelineResponse, ExportPipelineDSLRequest, ExportPipelineDSLResponse, - ImportPipelineDSLConfirmRequest, ImportPipelineDSLConfirmResponse, ImportPipelineDSLRequest, ImportPipelineDSLResponse, @@ -95,12 +94,12 @@ export const useImportPipelineDSL = ( } export const useImportPipelineDSLConfirm = ( - mutationOptions: MutationOptions = {}, + mutationOptions: MutationOptions = {}, ) => { return useMutation({ mutationKey: [NAME_SPACE, 'dsl-import-confirm'], - mutationFn: (request: ImportPipelineDSLConfirmRequest) => { - return post('/rag/pipeline/import/confirm', { body: request }) + mutationFn: (importId: string) => { + return post(`/rag/pipeline/imports/${importId}/confirm`) }, ...mutationOptions, })