-
Notifications
You must be signed in to change notification settings - Fork 15.4k
Add Patch Task Instance Summary #50526
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
Add Patch Task Instance Summary #50526
Conversation
3f37926
to
d31279c
Compare
Following discussion with Brent, we should probably also support task group there. |
When we will want to incorporate task groups we will encounter the same problem, we will not be able to update the I feel like maybe we update it (change the return type) as a bugfix because we will not be able to handle Note to self: |
I'm ok with calling the change a bug fix. We supported this in the old UI but it fell through the cracks in 3.0.0 |
#50550 PR for the endpoint spec fix. That can be targeted for |
Do not merge, need to be updated once #50550 is merged |
d31279c
to
b76da2e
Compare
b76da2e
to
836cabf
Compare
I need to update the PR description before merging |
* Add Patch Task Instance Summary * Test small adjustment
related to: #48584
Update the
patch_task_instance_dry_run
andpatch_task_instance endpoints to handle
ti summaries(i.e mapped TI without specifying the map_index => all of them will be affected). Default value is changed to
Nonebut it will not break because it supersede the
-1` for unmapped tasks.