Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

Support for Expressions on Script EIP #547

Closed
Delawen opened this issue Mar 17, 2023 · 2 comments · Fixed by #565
Closed

Support for Expressions on Script EIP #547

Delawen opened this issue Mar 17, 2023 · 2 comments · Fixed by #565
Assignees
@igarashitm
Labels
bug Something isn't working

Comments

@Delawen
Copy link
Member

Delawen commented Mar 17, 2023

After #541 we are dropping support of expressions in favor of languages.

Reenable it as in

https://github.com/apache/camel/blob/main/dsl/camel-yaml-dsl/camel-yaml-dsl/src/test/groovy/org/apache/camel/dsl/yaml/ScriptTest.groovy
@Delawen Delawen added the enhancement New feature or request label Mar 17, 2023
@igarashitm igarashitm self-assigned this Mar 17, 2023
@igarashitm igarashitm mentioned this issue Mar 17, 2023
Merged
@igarashitm
Copy link
Contributor

Not dropping acutally, since Script step doesn't work at all before that. It has properties object as a step parameter which doesn't make sense. See screenshot in #540 (comment)

@igarashitm igarashitm added bug Something isn't working and removed enhancement New feature or request labels Mar 17, 2023
@igarashitm igarashitm mentioned this issue Mar 17, 2023
Open
@igarashitm igarashitm removed their assignment Mar 17, 2023
@Delawen
Copy link
Member Author

Delawen commented Mar 20, 2023

This may be part of #516

@igarashitm igarashitm self-assigned this Mar 21, 2023
@igarashitm igarashitm mentioned this issue Mar 22, 2023
Merged
Delawen pushed a commit that referenced this issue Mar 23, 2023
@igarashitm @Delawen
fix: expression object support
6bb67a0 
Fixes: #547
Fixes: #548
Fixes: #554
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@igarashitm igarashitm

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: expression object support
2 participants
@igarashitm @Delawen