Closed lokanandaprabhu closed 1 week ago
@lokanandaprabhu: This pull request references SRVKP-6739 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.18.0" version, but no target version was set.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: lokanandaprabhu
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/uncc @saumeya /cc @Lucifergene
/cherrypick release-1.16.x
@lokanandaprabhu: cannot checkout release-1.16.x
: error checking out "release-1.16.x": exit status 1 error: pathspec 'release-1.16.x' did not match any file(s) known to git
/cherry-pick release-v1.16.x
@lokanandaprabhu: #183 failed to apply on top of branch "release-v1.16.x":
Applying: handled Unable to remove finally tasks in pipeline builder mode issue
Using index info to reconstruct a base tree...
A src/components/pipeline-builder/utils.ts
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): src/components/pipeline-builder/utils.ts deleted in HEAD and modified in handled Unable to remove finally tasks in pipeline builder mode issue. Version handled Unable to remove finally tasks in pipeline builder mode issue of src/components/pipeline-builder/utils.ts left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 handled Unable to remove finally tasks in pipeline builder mode issue
Pipeline builder was migrated in Pipelines release 1.17
So no need to migrate to 1.16.x branch
Fixes: https://issues.redhat.com/browse/SRVKP-6739
Analysis / Root cause: If all Finally tasks are removed, existing Pipeline Finally task was considered, so user was not able to remove all the Finally tasks
Solution Description: Used finallyTasks from formData
Screen shots / Gifs for design review:
----BEFORE---
https://github.com/user-attachments/assets/2fb44ca2-1be6-44a9-9bad-b5337c151fed
---AFTER----
https://github.com/user-attachments/assets/07f9fda8-dfff-4942-be3a-b91dd1f474da
Unit test coverage report: NA
Test setup:
Browser conformance: