Skip to content

docs(modeler): updated the docs to reflect the variableNames changes#8061

Open
giorgionaps wants to merge 3 commits intomainfrom
conditional-events-update
Open

docs(modeler): updated the docs to reflect the variableNames changes#8061
giorgionaps wants to merge 3 commits intomainfrom
conditional-events-update

Conversation

@giorgionaps
Copy link
Contributor

@giorgionaps giorgionaps commented Feb 25, 2026

Description

Doc updates that reflect the decision to remove the variableNames filter and add auto variable name filter detection for conditional events.

When should this change go live?

  • This is a bug fix, security concern, or something that needs urgent release support. (add bug or support label)
  • This is already available but undocumented and should be released within a week. (add available & undocumented label)
  • This is on a specific schedule and the assignee will coordinate a release with the Documentation team. (create draft PR and/or add hold label)
  • This is part of a scheduled alpha or minor. (add alpha or minor label)
  • There is no urgency with this change (add low prio label)

PR Checklist

  • My changes are for an upcoming minor release and are in the /docs directory (version 8.9).
  • My changes are for an already released minor and are in a /versioned_docs directory.

@github-actions
Copy link
Contributor

👋 🤖 🤔 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/.

  • docs/components/concepts/conditionals.md
  • docs/components/modeler/bpmn/conditional-events/conditional-events.md

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.


### Expression-based subscriptions

When a conditional event is activated, the engine analyzes its FEEL condition and derives which variables
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the spacing is a little off here, no?

#### Variable filter semantics

Conditional events can define variable filters to limit when the engine re-evaluates the condition.
By default, the engine derives the set of variables that can trigger an event from the FEEL expression.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here, shouldn't this be a paragraph as opposed to several separate lines?

Define variable filters unless you intentionally want the event to be evaluated on every variable change while the condition is satisfied.

Without a filter, a conditional event whose condition evaluates to `true` can trigger repeatedly.
Even with expression-based dependencies and optional filters, a conditional event whose condition evaluates to `true` can trigger repeatedly while it remains `true` (for example, for non-interrupting events).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here.

Copy link
Contributor

@christinaausley christinaausley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't want to flag every instance, but the line spacing appears to be off, perhaps from copilot or copy/paste issues? I would likely also suggest turning a series of sentences on separate lines into clean, concise paragraphs where possible. Let me know if the spacing was intentional for another reason but otherwise would clear up this whitespace 👍

The build is also failing, so not sure if you just need to re-run or merge in main to resolve. I also didn't see an issue linked, not sure if this is tracked in a particular ticket or if we need to create one? Let me know if you need any help!

Consolidate multiple lines into single sentences for clarity.
@giorgionaps giorgionaps added the hold This issue is parked, do not merge. label Feb 25, 2026
@giorgionaps
Copy link
Contributor Author

I didn't want to flag every instance, but the line spacing appears to be off, perhaps from copilot or copy/paste issues? I would likely also suggest turning a series of sentences on separate lines into clean, concise paragraphs where possible. Let me know if the spacing was intentional for another reason but otherwise would clear up this whitespace 👍

The build is also failing, so not sure if you just need to re-run or merge in main to resolve. I also didn't see an issue linked, not sure if this is tracked in a particular ticket or if we need to create one? Let me know if you need any help!

@christinaausley there seemed to be an issue indeed. Not sure what happened there as it looked normal in my vs code setup haha. Thanks for pointing that out, should be fixed now. 👌
There is no issue for this update atm, do you think it could be useful to create one for this change?


The semantics described below apply to conditional events within running process instances.
For process-level conditional start events, see [Trigger root-level conditional start events via API](#trigger-root-level-conditional-start-events-via-api).
The semantics described below apply to conditional events within running process instances. For process-level conditional start events, see [Trigger root-level conditional start events via API](#trigger-root-level-conditional-start-events-via-api).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The semantics described below apply to conditional events within running process instances. For process-level conditional start events, see [Trigger root-level conditional start events via API](#trigger-root-level-conditional-start-events-via-api).
The semantics described below apply to conditional events within running process instances. For process-level conditional start events, see [trigger root-level conditional start events via API](#trigger-root-level-conditional-start-events-via-api).

Nitpick 😄


In addition to scope activation, conditional events can also be triggered when relevant variables change within the event’s visible scope.
When a variable changes (for example, it is created or updated), the engine evaluates the condition for any active conditional events in the variable's scope and all child scopes (see [Top-down evaluation](#top-down-evaluation)).
When a variable changes (for example, it is created or updated), the engine evaluates the condition for any active conditional events in the variable's scope and all child scopes whose condition depends on that variable (see [Top-down evaluation](#top-down-evaluation)).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
When a variable changes (for example, it is created or updated), the engine evaluates the condition for any active conditional events in the variable's scope and all child scopes whose condition depends on that variable (see [Top-down evaluation](#top-down-evaluation)).
When a variable changes (for example, it is created or updated), the engine evaluates the condition for any active conditional events in the variable's scope and all child scopes whose condition depends on that variable (see [top-down evaluation](#top-down-evaluation)).

If the process instance updates variable `x` to `11` (for example, via the [Update element instance variables API](../../apis-tools/orchestration-cluster-api-rest/specifications/create-element-instance-variables.api.mdx)), the engine evaluates the condition for the conditional start event in the event subprocess.
Since the condition is now `true`, the conditional start event triggers and the event subprocess starts.
An event subprocess with a conditional start event is defined with condition `x > 10`. Assume the process instance starts with variable `x` initialized to `5`. The main process starts and executes the service task, while the event subprocess is not triggered because the condition is not satisfied.
If the process instance updates variable `x` to `11` (for example, via the [Update element instance variables API](../../apis-tools/orchestration-cluster-api-rest/specifications/create-element-instance-variables.api.mdx)), the engine evaluates the condition for the conditional start event in the event subprocess. Since the condition is now `true`, the conditional start event triggers and the event subprocess starts.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
If the process instance updates variable `x` to `11` (for example, via the [Update element instance variables API](../../apis-tools/orchestration-cluster-api-rest/specifications/create-element-instance-variables.api.mdx)), the engine evaluates the condition for the conditional start event in the event subprocess. Since the condition is now `true`, the conditional start event triggers and the event subprocess starts.
If the process instance updates variable `x` to `11` (for example, via the [update element instance variables API](../../apis-tools/orchestration-cluster-api-rest/specifications/create-element-instance-variables.api.mdx)), the engine evaluates the condition for the conditional start event in the event subprocess. Since the condition is now `true`, the conditional start event triggers and the event subprocess starts.

Copy link
Contributor

@christinaausley christinaausley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would just adjust links across both files so they don't capitalize the first word, [this for example]() instead of [This for example]() 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.9.0-alpha5 hold This issue is parked, do not merge.

Projects

Status: 👀 In Review

Development

Successfully merging this pull request may close these issues.

2 participants