mirror of https://github.com/langgenius/dify.git
fix(workflow): Remove duplicate Start node validation
- Skip Start node requirement in isRequiredNodesType loop since it's already covered by unified entry node validation - Eliminates duplicate 'User Input must be added' error when trigger nodes are present - Both useChecklist and useChecklistBeforePublish now consistently handle entry node validation - Resolves UI showing redundant validation errors for Start vs Trigger nodes
This commit is contained in:
parent
2dca0c20db
commit
08e8f8676e
|
|
@ -173,6 +173,10 @@ export const useChecklist = (nodes: Node[], edges: Edge[]) => {
|
|||
const isRequiredNodesType = Object.keys(nodesExtraData!).filter((key: any) => (nodesExtraData as any)[key].metaData.isRequired)
|
||||
|
||||
isRequiredNodesType.forEach((type: string) => {
|
||||
// Skip Start node requirement since we already check for any start/trigger nodes above
|
||||
if (type === BlockEnum.Start)
|
||||
return
|
||||
|
||||
if (!filteredNodes.find(node => node.data.type === type)) {
|
||||
list.push({
|
||||
id: `${type}-need-added`,
|
||||
|
|
@ -335,6 +339,11 @@ export const useChecklistBeforePublish = () => {
|
|||
|
||||
for (let i = 0; i < isRequiredNodesType.length; i++) {
|
||||
const type = isRequiredNodesType[i]
|
||||
|
||||
// Skip Start node requirement since we already check for any start/trigger nodes above
|
||||
if (type === BlockEnum.Start)
|
||||
continue
|
||||
|
||||
if (!filteredNodes.find(node => node.data.type === type)) {
|
||||
notify({ type: 'error', message: t('workflow.common.needAdd', { node: t(`workflow.blocks.${type}`) }) })
|
||||
return false
|
||||
|
|
|
|||
Loading…
Reference in New Issue