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.

Sign up for GitHub

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

Jump to bottom

Support converging inclusive gateway #3613

Closed
koevskinikola opened this issue May 16, 2023 · 8 comments · Fixed by #3746 or #4411
Closed

Support converging inclusive gateway #3613

koevskinikola opened this issue May 16, 2023 · 8 comments · Fixed by #3746 or #4411
Assignees
@barmac
Labels
Camunda 8 Flags an issue as related to Camunda 8 enhancement New feature or request
Milestone
M79

Comments

@koevskinikola
Copy link
Member

Problem you would like to solve

Support for the converging inclusive gateway (i.e. converging OR gateway) is added to Zeebe in camunda/camunda#12642 as an external contribution. Of course, all the Camunda Platform 8 need to be aligned and support the feature.

If the Modeler doesn't support modeling the converging inclusive gateway for Camunda Platform 8, then our customers cannot use this new feature.

Proposed solution

Please support the converging inclusive gateway for Camunda Platform 8. The behavior is specified in the related Zeebe issue.

  • Remove the validation that restricts the number of incoming sequence flows to an inclusive gateway to max 1.

Alternatives considered

  • Users can update the XML by hand
  • Users can use combinations of XOR and AND gateways to rebuild the OR gateway logic.

Additional context
@koevskinikola koevskinikola added the enhancement New feature or request label May 16, 2023
@nikku nikku added this to the 8.3 milestone May 16, 2023
@nikku nikku added the backlog Queued in backlog label May 16, 2023
@jschulenklopper jschulenklopper mentioned this issue Jun 27, 2023
Closed
@nikku nikku added ready Ready to be worked on and removed backlog Queued in backlog labels Jul 17, 2023
@nikku
Copy link
Member

nikku commented Jul 17, 2023

Moving to ready as this is now implemented in Zeebe.

@nikku nikku added the Camunda 8 Flags an issue as related to Camunda 8 label Jul 17, 2023
barmac added a commit to camunda/bpmnlint-plugin-camunda-compat that referenced this issue Jul 25, 2023
@barmac
feat: support converging inclusive gateway in 8.3
cfae72c 
Related to camunda/camunda-modeler#3613
@barmac barmac mentioned this issue Jul 25, 2023
Merged
@barmac
Copy link
Collaborator

barmac commented Jul 25, 2023

This will be implemented upstream via camunda/bpmnlint-plugin-camunda-compat#111

barmac added a commit to camunda/bpmnlint-plugin-camunda-compat that referenced this issue Jul 25, 2023
@barmac
feat: support converging inclusive gateway in 8.3
b6506b7 
Related to camunda/camunda-modeler#3613
merge-me bot pushed a commit to camunda/bpmnlint-plugin-camunda-compat that referenced this issue Jul 25, 2023
@barmac @merge-me
feat: support converging inclusive gateway in 8.3
7ed5bb3 
Related to camunda/camunda-modeler#3613
@barmac
Copy link
Collaborator

barmac commented Jul 25, 2023

Fixed upstream via camunda/bpmnlint-plugin-camunda-compat#111

@barmac barmac added the fixed upstream Requires integration of upstream change label Jul 25, 2023 — with bpmn-io-tasks
@barmac barmac removed the ready Ready to be worked on label Jul 25, 2023
barmac added a commit that referenced this issue Jul 25, 2023
@barmac
deps: update to @camunda/[email protected]
5eaf81b 
Closes #3613
Closes #3712
Closes #3577
@bpmn-io-tasks bpmn-io-tasks bot added in progress Currently worked on and removed fixed upstream Requires integration of upstream change labels Jul 25, 2023
@barmac barmac mentioned this issue Jul 25, 2023
Merged
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Jul 25, 2023
barmac added a commit that referenced this issue Jul 25, 2023
@barmac
deps: update to @camunda/[email protected]
ee0d46b 
Closes #3613
Closes #3712
Closes #3577
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jul 25, 2023
@CatalinaMoisuc
Copy link
Member

CatalinaMoisuc commented Jul 26, 2023 •
edited
Loading

Note that this issue might need to be reopened due to the latest updates, more details in PHub epic here.

@barmac will leave it to you to reopen depending on the reply we receive then.

@barmac barmac mentioned this issue Jul 26, 2023
Closed
@barmac
Copy link
Collaborator

barmac commented Jul 26, 2023

As a consequence of the decision PHub epic, I am reopening this issue, moving to the backlog, and removing it from the 8.3 milestone.

@barmac barmac reopened this Jul 26, 2023
@barmac barmac removed their assignment Jul 26, 2023
@barmac barmac removed this from the 8.3 milestone Jul 26, 2023
@saig0
Copy link
Member

saig0 commented Jun 18, 2024

Update: The feature is implemented in Camunda for version 8.6. 🚀

@barmac please plan to support it in the Modeler for Camunda version 8.6.

@nikku nikku added ready Ready to be worked on and removed backlog Queued in backlog labels Jun 19, 2024
@nikku
Copy link
Member

nikku commented Jun 19, 2024

Moved to ready.

@nikku nikku added this to the 8.6 milestone Jun 19, 2024
@nikku nikku modified the milestones: 8.6, M79 Jun 27, 2024
@barmac barmac self-assigned this Jun 28, 2024
barmac added a commit to camunda/bpmnlint-plugin-camunda-compat that referenced this issue Jun 28, 2024
@barmac
feat: allow converging inclusive gateway in 8.6
75a3105 
Related to camunda/camunda-modeler#3613
@barmac barmac mentioned this issue Jun 28, 2024
Merged
barmac added a commit to camunda/bpmnlint-plugin-camunda-compat that referenced this issue Jun 28, 2024
@barmac
feat: allow converging inclusive gateway in 8.6
2c01efd 
Related to camunda/camunda-modeler#3613
nikku pushed a commit to camunda/bpmnlint-plugin-camunda-compat that referenced this issue Jul 1, 2024
@barmac @nikku
feat: allow converging inclusive gateway in 8.6
0c78389 
Related to camunda/camunda-modeler#3613
@barmac barmac added the fixed upstream Requires integration of upstream change label Jul 1, 2024 — with bpmn-io-tasks
@barmac barmac removed the ready Ready to be worked on label Jul 1, 2024
@barmac
Copy link
Collaborator

barmac commented Jul 1, 2024

Fixed upstream via camunda/bpmnlint-plugin-camunda-compat#166

nikku added a commit to camunda/linting that referenced this issue Jul 1, 2024
@nikku
deps: update to [email protected]
7465c5d 
Related to camunda/camunda-modeler#3613
@nikku nikku mentioned this issue Jul 1, 2024
Merged
barmac pushed a commit to camunda/linting that referenced this issue Jul 1, 2024
@nikku @barmac
deps: update to [email protected]
9094147 
Related to camunda/camunda-modeler#3613
barmac added a commit that referenced this issue Jul 1, 2024
@barmac
feat: support converging inclusive gateway
069414e 
Closes #3613

deps: update to `@camunda/[email protected]`
@bpmn-io-tasks bpmn-io-tasks bot added in progress Currently worked on and removed fixed upstream Requires integration of upstream change labels Jul 1, 2024
@barmac barmac mentioned this issue Jul 1, 2024
Merged
4 tasks
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Jul 1, 2024
@nikku nikku closed this as completed in 6102759 Jul 1, 2024
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@barmac barmac

Labels
Camunda 8 Flags an issue as related to Camunda 8 enhancement New feature or request
Projects
None yet
Milestone
M79
5 participants
@nikku @koevskinikola @saig0 @CatalinaMoisuc @barmac