Skip to content
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

It's not necessary to select myself when changing the next node. #4444

Closed
4 tasks done
HiChen404 opened this issue May 16, 2024 · 0 comments · Fixed by #4445
Closed
4 tasks done

It's not necessary to select myself when changing the next node. #4444

HiChen404 opened this issue May 16, 2024 · 0 comments · Fixed by #4445
Labels
💪 enhancement New feature or request

Comments

@HiChen404
Copy link
Contributor

Self Checks

  • I have searched for existing issues search for existing issues, including closed ones.
  • I confirm that I am using English to submit this report (我已阅读并同意 Language Policy).
  • Please do not modify this template :) and fill in all the required fields.

1. Is this request related to a challenge you're experiencing?

image

2. Describe the feature you'd like to see

The self node type item should not show in the picker when changing the next node.

3. How will this feature improve your workflow or experience?

Each time the next node is changed, it clears the next node content.
If 'self' is selected, it also clears the next node content.
The user should avoid making unnecessary node changes.

4. Additional context or comments

Another minor optimization unrelated to this issue:

When the node change, the handleNodeChange method is called, which removes all connectedEdges from the node.

I think the previous line should be kept because the line between the previous node and the next node is deterministic.

I'm a newbie and am learning dify, thanks.

5. Can you help us with this feature?

  • I am interested in contributing to this feature.
@dosubot dosubot bot added the 💪 enhancement New feature or request label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💪 enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant