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

Implement zeebe:taskDefinition binding #117

Merged
merged 3 commits into from
Nov 1, 2023

Conversation

barmac
Copy link
Contributor

@barmac barmac commented Oct 27, 2023

This adds zeebe:taskDefinition binding which allows to fully configure task definition:

{
  "binding": {
    "type": "zeebe:taskDefinition",
    "property": "retries"
  }
},
{
  "binding": {
    "type": "zeebe:taskDefinition",
    "property": "type"
  }
}

This PR also removes obsolete zeebe:taskDefinition:retries binding. The binding was never implemented in bpmn-js-element-templates. It is unnecessary with introduction of zeebe:taskDefinition binding.

BREAKING CHANGE: zeebe:taskDefinition:retries binding is removed.

Related to camunda/camunda-modeler#2936

The binding was never implemented in bpmn-js-element-templates.
It is unnecessary with introduction of `zeebe:taskDefinition` binding.

BREAKING CHANGE: `zeebe:taskDefinition:retries` binding is removed.
The binding is replaced with `zeebe:taskDefinition` which suggests
both `retries` and `type` property names.
@barmac barmac force-pushed the implement-zeebe-task-definition-binding branch from 99d84d5 to 70574dd Compare October 30, 2023 16:00
@barmac barmac requested review from a team, philippfromme and marstamm and removed request for a team October 31, 2023 17:31
@barmac barmac marked this pull request as ready for review October 31, 2023 17:31
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Oct 31, 2023
@marstamm marstamm merged commit a822938 into main Nov 1, 2023
2 checks passed
@marstamm marstamm deleted the implement-zeebe-task-definition-binding branch November 1, 2023 12:46
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Nov 1, 2023
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.

2 participants