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

Multiselect for instance deletion #3268

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

wuahi
Copy link

@wuahi wuahi commented Apr 8, 2022

Check List:

  • Unit tests: NO
  • Documentation: NO
  • Added the possibility to multiselect instances under Admin app
  • Added the possibility to delete multiple instances at a time from the instances listing. They are first terminated with a set reason, and then deleted.
  • List of failed deletions , links to failed processes that open in a new window

flowable-process-instances

@wuahi wuahi marked this pull request as ready for review April 8, 2022 12:19
@wuahi
Copy link
Author

wuahi commented May 3, 2022

Is there something i need to do in before a maintainer will take a look at this pr?

@tijsrademakers
Copy link
Contributor

Hi @wuahi ,

We are currently looking at extending the REST API with some bulk action support including terminating / deleting process instances. When this would be available no loop of calling the REST API per process instance from the Admin web application is needed anymore. So we would first like to explore the bulk action support in the REST API a bit more.

Best regards,

Tijs

@wuahi
Copy link
Author

wuahi commented May 3, 2022

Thanks for the quick response and this is lovely to hear! Any time estimate for this extension of the REST API?

@filiphr
Copy link
Contributor

filiphr commented Jul 11, 2022

Hey @wuahi, we have added this functionality on main.

The endpoint for deleting process instances is here and the PR that added this functionality is #3295.

Can you please rebase your branch on main and adapt your changes to use the new REST API?

@wuahi
Copy link
Author

wuahi commented Jul 14, 2022

Hey @wuahi, we have added this functionality on main.

The endpoint for deleting process instances is here and the PR that added this functionality is #3295.

Can you please rebase your branch on main and adapt your changes to use the new REST API?

Nice to hear! Yeah sure. I will try to do it within a week. Thanks for the info :)

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.

None yet

4 participants