inline hinting for ExecuteRequest#287
Conversation
Match the capabilities of GetActionRequest to support hints for composing ActionResults for Execute
EdSchouten
left a comment
There was a problem hiding this comment.
Instead of doing this, what are our thoughts on having a nested message for specifying all these hints, so that it's usable from within both GetActionResultRequest and ExecuteRequest?
Conceptually that's fine, but given that the messages already exist in one place, I think it's better to just duplicate them here than to force another migration from moving fields around. The practical implication of switching to a nested message would be that clients end up needing to set both fields in GetActionRequest indefinitely. |
|
@bergsieker good for a merge? no privs here |
Match the capabilities of GetActionRequest to support hints for composing ActionResults for Execute