-
Notifications
You must be signed in to change notification settings - Fork 137
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
Document configuration options #688
Comments
Consider the good practice we talked about in the context of bpmn-js: Have a dedicated |
Link that example from the readme, where appropriate. |
Hi @barmac, @nikku, @pinussilvestrus, Is there a simple example somewhere for those configuration options? Extra question: would these things be configured differently under dmn-js 11 (we haven't braved the breaking changes in version 12 yet) or is the configuration object the same then? |
Data types:
Expression languages: dmn-js/packages/dmn-js-shared/src/features/expression-languages/ExpressionLanguages.js Line 21 in 7214517
It hasn't changed since it was implemented. |
Okay, cool. Thanks for that. new DmnJS({
common: {
dataTypes: [
'string',
'boolean',
'integer',
'long',
'double',
'date'
],
expressionLanguages: {
options: [
{ value: 'juel', label: 'JUEL' },
{ value: 'javascript', label: 'JavaScript' },
{ value: 'feel', label: 'FEEL' }
],
defaults: {
inputCell: 'feel',
outputCell: 'juel',
editor: 'juel'
}
}
}
}) If I understand it correctly, the One extra question though: how to differentiate between an input expression (i.e., in the input column header, that in the past had "juel" as default) vs. an input cell (that in the past had "feel" as default already)? |
Correct.
If your input expression should have a different default than the editor, you would need to build a custom extension. We don't support that finely grained configuration. Also, what decision engine do you use? |
We're using the Camunda decision engine for years now.
We are just trying to recreate the same behaviour (in order to ensure backward compatibility for the hundreds of stored decision tables) when upgrading to a more recent version of both the engine and the editor component. |
Indeed, the configuration you provided should maintain the legacy defaults. If you ever plan to migrate to the new defaults, this will require batch change of the diagrams to set the expression language property for non-FEEL expressions. |
What should we do?
With dmn-js 12, the library will expose at least two configuration options:
Let's document this is currently everything is burried in the source code. This could be part of the
usage
section of the readme.Why should we do it?
Let's make the features discoverable.
The text was updated successfully, but these errors were encountered: