This PR fully onboards the get workflow API. Note we do the same logic as searchWorkflows(), in that we make an additional call to fetch the workflow state as well. This is to prevent bugs or edge cases where everything is present except state, and less calls needing to be made from client -> server.
Also:
audits existing TODOs and removes irrelevant ones
minor refactoring of toWorkflowObj() to reuse it in getWorkflow()
Check List
[x] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.
Description
This PR fully onboards the get workflow API. Note we do the same logic as
searchWorkflows(), in that we make an additional call to fetch the workflow state as well. This is to prevent bugs or edge cases where everything is present except state, and less calls needing to be made from client -> server.Also:
toWorkflowObj()to reuse it ingetWorkflow()Check List
--signoffBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.