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.

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

Feat/json in pebble #4289

Merged
merged 3 commits into from
Jul 10, 2024
Merged

Feat/json in pebble #4289

merged 3 commits into from
Jul 10, 2024

Conversation

loicmathieu
Copy link
Member

Fixes #3715

Example flow:

id: ion
namespace: company.myteam
tasks:
  - id: output
    type: io.kestra.plugin.core.output.OutputValues
    values:
      data.json: "{{ ionDecode('{date:2024-04-21T23:00:00.000Z, title:\"Main_Page\",views:109787}').title }}"

@loicmathieu loicmathieu added the kind/breaking-change Code modifications needed before upgrading label Jul 9, 2024
@loicmathieu
Copy link
Member Author

loicmathieu commented Jul 9, 2024

Breaking Changes

The json Pebble expression filter has been renamed to jsonDecode, using the old filter will emit a warning in the server log.
The json() Pebble expression function has been renamed to jsonEncode, using the old function will emit a warning in the server log.

Copy link

sonarcloud bot commented Jul 10, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
14.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@loicmathieu loicmathieu merged commit fc506c4 into develop Jul 10, 2024
8 of 9 checks passed
@loicmathieu loicmathieu deleted the feat/json-in-pebble branch July 10, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/breaking-change Code modifications needed before upgrading
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Improve support for ION/JSON processing in Pebble expressions
1 participant