Skip to content

solve unsafe usage #5066

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Chononut
Copy link

参数change是NodeChange类型,Nodechange是6个类型的联合类型。只有当6个类型同时拥有同一属性时,TS才能找到这一属性。6个类型只有NodePositionChange有dragging属性,在使用change.dragging之前需要使用条件语句类型收缩。

Copy link
Contributor

gru-agent bot commented Jun 18, 2025

TestGru Assignment

Summary

Link CommitId Status Reason
Detail c4bec83 🚫 Skipped No files need to be tested {"projects/app/src/pageComponents/app/detail/WorkflowComponents/Flow/hooks/useWorkflow.tsx":"File path does not match include patterns."}

History Assignment

Tip

You can @gru-agent and leave your feedback. TestGru will make adjustments based on your input

Copy link

cla-assistant bot commented Jun 18, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

cla-assistant bot commented Jun 18, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

Preview mcp_server Image: ghcr.io/labring/fastgpt-pr:fatsgpt_mcp_server_c4bec8345151e3ab8c6408128ed726286e2fb719

Copy link

Preview sandbox Image: ghcr.io/labring/fastgpt-pr:fatsgpt_sandbox_c4bec8345151e3ab8c6408128ed726286e2fb719

Copy link

Preview fastgpt Image: ghcr.io/labring/fastgpt-pr:fatsgpt_c4bec8345151e3ab8c6408128ed726286e2fb719

@c121914yu
Copy link
Collaborator

c121914yu commented Jun 18, 2025

没看懂有啥区别,不是已经推断出来了么?
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants