Closed marcelklehr closed 2 months ago
Attention: Patch coverage is 39.13043%
with 14 lines
in your changes missing coverage. Please review.
Please upload report for BASE (
main@6b0f960
). Learn more about missing BASE report. Report is 2 commits behind head on main.
requires Nextcloud 30
So this can only be part of the next planned major version 5.0 as we still support Nextcloud 29 an 28, while there are already plans for 5.0 we still need to work on some features / technical dept.
I removed the requirement of Nextcloud 30, so we can hopefully merge this sooner :pray:
@marcelklehr nice :) please fix the errors, then we can merge it :)
@come-nc @marcelklehr Can this be used to solve issues #1850, #1772, #1748, #1223 and/or #805?
Yes and no, this is about windmill webhooks based workflows. That allows to do even more than with Nextcloud flow but requires you to also host Windmill, so this is nothing that can be easily done on "plain" hosting.
Meaning the first 3 issues would I still consider feature requests, similar to the analytics integration - if someone wants to create an integration (and add proper tests) we could integrate that. The last one ... well the new windmill integration is called "flow" but that ticket is more about workflowengine flow which is the Nextcloud built-in alternative that does not require external API, maybe still something to consider but would also need work on our side.
only #1223 could be done with Nextcloud 30 due to the webhook_listeners
app integrated with Nextcloud now.
You should be able to register webhooks based on the forms submitted event. I did not test this yet, but probably something we could document and then close the request :)
requires Nextcloud 30
This will allow users of the upcoming windmill workflow automation app to trigger automations based on form submissions.