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
Fixes #11616 - Adding job invocation api controller #17
Conversation
[test] |
f5d580d
to
e7d3aac
Compare
process_response @composer.save! | ||
end | ||
|
||
def resource_name |
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.
is this needed? the controller name seems standard
e7d3aac
to
106fbb3
Compare
106fbb3
to
830578e
Compare
830578e
to
9233fc3
Compare
307c968
to
4a3b055
Compare
@@ -0,0 +1,3 @@ | |||
collection @job_templates |
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.
@job_invocations
, unfortunately functional tests don't cover this
4a3b055
to
9dfaa51
Compare
rebase wanted |
5c0d58b
to
854a12f
Compare
rebased! |
854a12f
to
da4bb05
Compare
rebased again |
[test] |
Tested again, works nicely, ACK => merge at will |
We agreed with @iNecas that there's no need to wait for author to merge his own PR after it's ACKed, so I'm merging this one. Thanks @jlsherrill! |
Fixes #11616 - Adding job invocation api controller
No description provided.