search

camunda / bpmnlint-plugin-camunda-compat

A bpmnlint plug-in for Camunda BPMN compatibility.
MIT License
0 stars 2 forks source link

deps: integrate `@bpmn-io/feel-lint@1.2.0` #153

Closed marstamm closed 10 months ago

marstamm commented 10 months ago

This PR integrates latest FEEL linter changes and restores the 1.1.0 Behavior of overriding the supplied error message. The Detailes message will be shown in the editor:

image

related to https://github.com/camunda/camunda-modeler/issues/3991

marstamm commented 10 months ago

In the error panel, the errors are grouped by Activity. When you have multiple Errors in the same Task, it should be clear that multiple properties are affected. image

This is in line with all other Error messages. If we want to change the error message in the Properties panel, this should be done in the @camunda/linting repository. https://github.com/camunda/linting/blob/68c0d021de459f00a78ea9480ab3e8feec2acbc0/lib/utils/properties-panel.js#L317-L319

nikku commented 10 months ago

Let's follow up with the updates there then. Thanks for clarifying.

marstamm commented 10 months ago

Follow-up here: https://github.com/camunda/linting/pull/97