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

Question : Can Start Node Accept Incoming Connections ? #1367

Open
satti-hari-krishna-reddy opened this issue Apr 9, 2024 · 1 comment
Open

Comments

@satti-hari-krishna-reddy
Copy link
Contributor

satti-hari-krishna-reddy commented Apr 9, 2024

Describe the Issue
Start Node is not supposed to have any arrows point to it but we can do something to make the arrow pointing towards start node

To Reproduce
Steps to reproduce the behavior:

  1. create a new workflow and drag two apps
  2. make the start node to point 2 nd app
  3. Now make the 2nd app as start Node

Expected behavior
That Should'nt be allowed as it has incoming arrow

Screenshots
Screenshot (95)

Screenshot (96)

@frikky
Copy link
Member

frikky commented Apr 21, 2024

Describe the Issue Start Node is not supposed to have any arrows point to it but we can do something to make the arrow pointing towards start node

To Reproduce Steps to reproduce the behavior:

  1. create a new workflow and drag two apps
  2. make the start node to point 2 nd app
  3. Now make the 2nd app as start Node

Expected behavior That Should'nt be allowed as it has incoming arrow

Screenshots Screenshot (95)

Screenshot (96)

To add to this:

  • The startnode SHOULD be able to have nodes pointing to it. This is because you may start from different positions at different times
  • The startnode should NOT interpret incoming branches however, as they may have conditions & parent node requirements that don't make sense
  • Triggers point to startnodes.

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

No branches or pull requests

2 participants