-
Notifications
You must be signed in to change notification settings - Fork 18
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
Eng 2702 Implement v2 workflow metadata routes #1151
Eng 2702 Implement v2 workflow metadata routes #1151
Conversation
…g-2698-m1-implement-v2-api-framework
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a deprecated handler, right? Is there some way we can differentiate it from the v2 handlers? Maybe either appending deprecated to this or v2 to the new ones.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that's a good idea!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually this file shouldn't be included, removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a couple minor comments, otherwise looks great! I like the clean implementation for generating the response objects 😎
…g-2702-m1-implement-workflow-metadata-routes
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
…g-2702-m1-implement-workflow-metadata-routes
Describe your changes and why you are making these changes
This PR implements 2 workflow metadata routes:
dag_get
anddag_result_get
. Besides following standard RTK boilerplate introduced in #1148 , we added a few adjustments:response
package so that it's easier to maintain 1-1 mapping against UI codesdeprecated
.Related issue number (if any)
ENG-2702
Tests
Compiles
Manual QA WIP
Checklist before requesting a review
python3 scripts/run_linters.py -h
for usage).run_integration_test
: Runs integration testsskip_integration_test
: Skips integration tests (Should be used when changes are ONLY documentation/UI)