Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Feature/webhook #222

Closed
wants to merge 25 commits into from
Closed

Feature/webhook #222

wants to merge 25 commits into from

Conversation

charybr
Copy link
Contributor

@charybr charybr commented May 16, 2023

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • Other (please describe):

Changes in this PR

Note: Work In Progress, will notify when it is ready for review.

Webhooks for workflow and task status
Problem:
We have a requirement to publish status of workflow and task to a Webhook.

Solution:
WorkflowStatusListener and TaskStatusListener interfaces can be implemented to call webhooks when there is a change in status of workflow and task. We can add new module called 'webhook'.

Issue #
#212

Alternatives considered

Describe alternative implementation you have considered

@egandro
Copy link

egandro commented Aug 15, 2023

This looks great. Any progress here?

@charybr charybr closed this Sep 28, 2023
@charybr
Copy link
Contributor Author

charybr commented Sep 28, 2023

Closing this PR, as these changes are already in another related PR - #228

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants