-
Notifications
You must be signed in to change notification settings - Fork 228
docs(modeler): conditional events #7908
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
base: main
Are you sure you want to change the base?
Conversation
|
👋 🤖 🤔 Hello, @giorgionaps! Did you make your changes in all the right places? These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.8/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
christinaausley
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! I have given your original doc a technical review, but felt like the lengthier bits of text could be broken into something slightly more readable/consolidated. There are of course many ways to do this, and I wanted to largely leave it up to you, so I created a new file conditional-events-new.md which can be seen in the deploy environment.
In this new file, I've broken up larger chunks of text into a tabbed approach and consolidated things a bit. Feel free to take a look, and you can replace the original conditional-events.md file with this if you want (you'll just need to remove the "new" portion added to the filename, ID, and page title). If you want to go a different direction, you can of course just delete this new file.
@christinaausley thank you so much for taking the time to reorganize the info, I will take a look at that file & see what's best to use here 🙌 |
| <bpmn:conditionalEventDefinition id="ConditionalEventDefinition_1"> | ||
| <bpmn:condition xsi:type="bpmn:tFormalExpression">= x > 1</bpmn:condition> | ||
| <bpmn:extensionElements> | ||
| <zeebe:conditionalFilter | ||
| variableNames="var1, var2" | ||
| variableEvents="create, update" /> | ||
| </bpmn:extensionElements> | ||
| </bpmn:conditionalEventDefinition> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🏅 Proper nesting is crucial to make the code readable.
|
|
||
| If you omit `zeebe:conditionalFilter`, the engine evaluates the condition on any variable change in the scope. This mirrors the default behavior of Camunda 7 conditional events, using FEEL conditions and Zeebe extensions in Camunda 8. | ||
|
|
||
| `variableEvents` is only supported for conditional events within a running scope (for example, intermediate conditional catch events, conditional boundary events, and conditional event subprocess start events). It is not supported for root-level conditional start events, as Camunda Modeler prevents or clears it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that we don't support variableEvents for process start event for a reason: There is no process instance in which the events related to the variable could happen yet. Variable events only make sense for an existing process instance.
|
✅ from my side |
|
🚧 The preview environment for the commit bf380e2 is being built. This usually takes 8-10 minutes. |
Description
Created a new doc for conditional events with examples
When should this change go live?
bugorsupportlabel)available & undocumentedlabel)holdlabel)low priolabel)PR Checklist
{type}(scope): {description}commit message(s)/docsdirectory (version 8.9)./versioned_docsdirectory.@camunda/tech-writersunless working with an embedded writer.