Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Invalid FEEL expression error is not displayed in the problems panel for execution listeners #4562

Closed
barmac opened this issue Sep 26, 2024 · 2 comments · Fixed by #4602
Closed

Invalid FEEL expression error is not displayed in the problems panel for execution listeners #4562

barmac opened this issue Sep 26, 2024 · 2 comments · Fixed by #4602
Assignees
@philippfromme
Labels
bug Something isn't working good first issue Good for newcomers linting spring cleaning Could be cleaned up one day
Milestone
M82

Comments

@barmac
Copy link
Collaborator

barmac commented Sep 26, 2024 •
edited by nikku
Loading

Describe the bug

Invalid FEEL expression error is not displayed in the problems panel for execution listeners

image

Steps to reproduce

  1. Create an execution listener
  2. Write an invalid FEEL expression
  3. Error indicated in properties panel, but not reflected in error panel ("problems tab")

Expected behavior

Error should be displayed in the problems panel.

Environment

  • OS: [e.g. MacOS 10.2, Windows 10]
  • Camunda Modeler Version: 5.27.0, 5.28.0
  • Execution Platform: Camunda 8
  • Installed plug-ins: [...]

Additional context

No response
@barmac barmac added bug Something isn't working ready Ready to be worked on good first issue Good for newcomers spring cleaning Could be cleaned up one day labels Sep 26, 2024
@nikku nikku mentioned this issue Sep 26, 2024
Closed
41 tasks
@nikku nikku added the linting label Sep 29, 2024
@philippfromme philippfromme added the in progress Currently worked on label Oct 10, 2024 — with bpmn-io-tasks
@philippfromme philippfromme removed the ready Ready to be worked on label Oct 10, 2024
philippfromme added a commit to camunda/bpmnlint-plugin-camunda-compat that referenced this issue Oct 10, 2024
@philippfromme
fix(feel): report errors for flow element containers
12e6e54 
Related to camunda/camunda-modeler#4562
@philippfromme philippfromme mentioned this issue Oct 10, 2024
Merged
philippfromme added a commit to camunda/bpmnlint-plugin-camunda-compat that referenced this issue Oct 10, 2024
@philippfromme
fix(feel): report errors for flow element containers
9256225 
Related to camunda/camunda-modeler#4562
philippfromme added a commit to camunda/linting that referenced this issue Oct 10, 2024
@philippfromme
deps: update bpmnlint-plugin-camunda-compat to v2.26.1
ab65875 
Related to camunda/camunda-modeler#4562
@philippfromme philippfromme mentioned this issue Oct 10, 2024
Merged
nikku pushed a commit to camunda/linting that referenced this issue Oct 10, 2024
@philippfromme @nikku
deps: update bpmnlint-plugin-camunda-compat to v2.26.1
2a22afb 
Related to camunda/camunda-modeler#4562
@barmac
Copy link
Collaborator Author

barmac commented Oct 11, 2024

This is fixed upstream and not in progress anymore: camunda/bpmnlint-plugin-camunda-compat#175

@barmac barmac added fixed upstream Requires integration of upstream change and removed in progress Currently worked on labels Oct 11, 2024
@philippfromme philippfromme mentioned this issue Oct 11, 2024
Merged
4 tasks
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed fixed upstream Requires integration of upstream change labels Oct 11, 2024
philippfromme added a commit that referenced this issue Oct 11, 2024
@philippfromme
deps(client): update @camunda/linting to v3.27.2
8aff5e9 
Closes #4562
barmac pushed a commit that referenced this issue Oct 14, 2024
@philippfromme @barmac
deps(client): update @camunda/linting to v3.27.2
105e3ff 
Closes #4562
@barmac
Copy link
Collaborator Author

barmac commented Oct 14, 2024

Closed via #4602

@barmac barmac closed this as completed Oct 14, 2024
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Oct 14, 2024
@github-actions github-actions bot added this to the M82 milestone Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philippfromme philippfromme

Labels
bug Something isn't working good first issue Good for newcomers linting spring cleaning Could be cleaned up one day
Projects
None yet
Milestone
M82
Development

Successfully merging a pull request may close this issue.

deps(client): update @camunda/linting to v3.27.2 camunda/camunda-modeler
3 participants
@nikku @philippfromme @barmac