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

Parameterless functions are suggested with non-existing parameter #4539

Closed
barmac opened this issue Sep 23, 2024 · 1 comment · Fixed by #4541
Closed

Parameterless functions are suggested with non-existing parameter #4539

barmac opened this issue Sep 23, 2024 · 1 comment · Fixed by #4541
Assignees
@nikku @barmac
Labels
bug Something isn't working DMN intelligence
Milestone
M81

Comments

@barmac
Copy link
Collaborator

barmac commented Sep 23, 2024

Describe the bug

It should suggest now().

image

Steps to reproduce

  1. Type "now"
  2. now this happens

Expected behavior

No parameters suggested.

Environment

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

Additional context

Reported already in bpmn-io/dmn-js#898
@barmac barmac added the bug Something isn't working label Sep 23, 2024
@barmac barmac self-assigned this Sep 23, 2024
@nikku
Copy link
Member

nikku commented Sep 23, 2024

Requires integration of https://github.com/bpmn-io/feel-editor/releases/tag/v1.9.1.

@nikku nikku added the fixed upstream Requires integration of upstream change label Sep 23, 2024
@nikku nikku mentioned this issue Sep 23, 2024
Closed
41 tasks
nikku added a commit that referenced this issue Sep 23, 2024
@nikku
fix: correct completion for non-arg FEEL built-ins
c55659a 
  Closes #4539

deps: bump to @bpmn-io/[email protected]
@nikku nikku mentioned this issue Sep 23, 2024
Merged
4 tasks
@bpmn-io-tasks bpmn-io-tasks bot added the needs review Review pending label Sep 23, 2024
@bpmn-io-tasks bpmn-io-tasks bot removed the fixed upstream Requires integration of upstream change label Sep 23, 2024
@nikku nikku added this to the M81 milestone Sep 23, 2024
@barmac barmac closed this as completed in 357f7ec Sep 23, 2024
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nikku nikku

@barmac barmac

Labels
bug Something isn't working DMN intelligence
Projects
None yet
Milestone
M81
Development

Successfully merging a pull request may close this issue.

Correct completion for non-arg FEEL built-ins camunda/camunda-modeler
2 participants
@nikku @barmac