-
Notifications
You must be signed in to change notification settings - Fork 227
Global user task listeners using unified configuration #7323
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?
Global user task listeners using unified configuration #7323
Conversation
|
👋 🤖 🤔 Hello, @fabiopaini-camunda! 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. |
fba03b7 to
c88c318
Compare
128a653 to
40c5cd4
Compare
c88c318 to
b3c6fd0
Compare
40c5cd4 to
c23bdef
Compare
b3c6fd0 to
c30af2c
Compare
c23bdef to
52661f5
Compare
c30af2c to
7a94471
Compare
korthout
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.
Awesome! LGTM 👍
The base branch was changed.
mesellings
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.
Approved 🚀
…, removing references to Unified Configuration" This reverts commit 52661f5.
aa47f58 to
bf5ed22
Compare
| | `retries` | No | Number of retries for the user task listener job. Defaults to `3` if not set. | | ||
| | `after-local` | No | Boolean indicating whether the listener should run after model-level listeners. Defaults to `false` (runs before model-level listeners). | | ||
| | Property | Required | Description | | ||
| |:-------------------| :------- | :-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | |
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.
[prettier] reported by reviewdog 🐶
| |:-------------------| :------- | :-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | |
| | :----------------- | :------- | :-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | |
Description
#7322 introduced documentation for the Global user task listeners feature as released with version 8.9.0-alpha1. The configuration has since been migrated to the Unified Configuration.
This PR fixes the documentation in order to provide the new correct configuration path.
Note: this should not be released before version 8.9.0-alpha2 to ensure that the currently-available configuration properties are shown in the public documentation.
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.