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

Addition of terminateRemove() Functions #9

Merged
merged 28 commits into from
Mar 27, 2024
Merged

Conversation

Doe1111
Copy link

@Doe1111 Doe1111 commented Mar 25, 2024

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • WHOSUSING.md
  • Other (please describe):

Changes

  • Added single terminate and remove feature
  • Added bulk terminate and remove feature

Ayush Thengne and others added 28 commits February 26, 2024 15:12
Initial commit to add bulk workflow deletion
Move bulk delete to impl that has executor with delete
Merging ayush-develop to main
Changed terminateRemove endpoint to use dashcase
Added terminateRemove() to listed file. Some potential changes to this commit to consider are removing the archive and/or reason params for the function if not needed.
@JeffP07 JeffP07 merged commit 04d3da0 into main Mar 27, 2024
JeffP07 added a commit that referenced this pull request Apr 14, 2024
* Initial commit to add bulk workflow deletion

* Added an API endpoint for bulk deletion

* Add archiveworkflow param

* Added bulk delete action to BulkActionModule

* Added archive option to bulk delete

* Addition of terminateRemove() Functions (#9)

* Added an API endpoint for bulk deletion

* Fixed WorkflowBulkServiceTest, added test cases for deleteWorkflow an… (#10)

* Fixed WorkflowBulkServiceTest, added test cases for deleteWorkflow and terminateRemove, and added WorkflowResourceTest test for single terminateRemove

* pass spotlessjava

---------

Co-authored-by: JeffP <[email protected]>

---------

Co-authored-by: jeffp1 <[email protected]>
Co-authored-by: Ayush Thengne <[email protected]>
Co-authored-by: Jeff P <[email protected]>
Co-authored-by: Doe1111 <[email protected]>
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