Skip to content
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

Refactor Task action resource to take in task definition uuid in addition to name #433

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gitcliff
Copy link
Contributor

@gitcliff gitcliff changed the title Refactor Task action resource to take in task definition uuid in addi… Refactor Task action resource to take in task definition uuid in addition to name May 17, 2020
@ibacher
Copy link
Member

ibacher commented May 19, 2020

@gitcliff Can you add a test showing that sending a POST to the TaskActionController2_4 with the payload:

{ "action": "runtask", "tasks": [" + getTaskUuid() + "] }

runs the task? This should be pretty similar to the existing tests for task name.

@tendomart
Copy link

@ibacher is everything here valid up to this level ,if some other person is to pick on this task ?

@ibacher ibacher force-pushed the master branch 2 times, most recently from f9c73bb to 2a4407f Compare December 14, 2020 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants