Conversation
| // Ref Value points to a Result of a node, the int is the id of the node | ||
| int64 result = 2; | ||
| // Points to Value that is contained inside of a node | ||
| InputType input_type = 3; |
There was a problem hiding this comment.
@nicosammito does this case exist? I thought we only had references to flow inputs or return values of nodes
There was a problem hiding this comment.
And also the references to the input of the flow itself. This is done by setting the nodeFunctionId to null. This is why we are making it optional on GraphQL
There was a problem hiding this comment.
Oh sorry some misunderstanding. For example this is used inside for Each. The item is an input type defined within the Consumer DataType. Therefore we have 3 reference variants:
The flow input itself, a return type of a node and the input types of a parameter of type node
Co-authored-by: Niklas van Schrick <mc.taucher2003@gmail.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>
| // Ref Value points to a Result of a node, the int is the id of the node | ||
| int64 result = 2; | ||
| // Points to Value that is contained inside of a node | ||
| InputType input_type = 3; |
There was a problem hiding this comment.
Oh sorry some misunderstanding. For example this is used inside for Each. The item is an input type defined within the Consumer DataType. Therefore we have 3 reference variants:
The flow input itself, a return type of a node and the input types of a parameter of type node
| // Database Identifier of node | ||
| oneof target { | ||
| // Ref Value points to flow input | ||
| int64 flow_input = 1; |
There was a problem hiding this comment.
Is this pointing to the flowId?
There was a problem hiding this comment.
No, it just be nothing, I thought we just always make this 0 but we can change it into something that represents empty
There was a problem hiding this comment.
yep I would prefer the empty solution
Co-authored-by: Nico Sammito <nico@sammito.de> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>
3 participants
Resolves: #211