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

Dil templates - Archive #58

Merged
merged 3 commits into from
Oct 26, 2022
Merged

Dil templates - Archive #58

merged 3 commits into from
Oct 26, 2022

Conversation

brunovg
Copy link
Collaborator

@brunovg brunovg commented Oct 21, 2022

dil template for Archive component

@brunovg brunovg requested a review from skin27 October 21, 2022 13:23
@brunovg brunovg self-assigned this Oct 21, 2022
@brunovg brunovg requested a review from skin27 October 24, 2022 10:07
Copy link
Member

@skin27 skin27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

.templateParameter("out")
.from("{{in}}")
.routeConfigurationId("{{routeconfiguration_id}}")
.to("dataformat:checkedZipFileDataFormat:marshal?usingIterator={{usingIterator}}")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • This statement only have a marshal. Is there is also an unmarshal?
  • Suggestion: Maken two kinds of routetemplates. zip (zip-action / zip-sink) and unzip (unzip-action / unzip-sink).
  • I noticed that above doesn't match the DIL file in custom-components / also missing archive (or zip / unzip in the dependencyutil).

@skin27 skin27 merged commit 55555b8 into develop Oct 26, 2022
@skin27 skin27 deleted the templates/archive branch October 26, 2022 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants