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

[craftercms] Bundle tool should allow users to choose what kind of archive they want to use. #1669

Closed
cortiz opened this Issue Dec 4, 2017 · 5 comments

Comments

@cortiz
Member

cortiz commented Dec 4, 2017

Expected behavior

crafter.bundle.archive=tar|zip should be added so user can choose the output archive,
if not send, the default value should be both (tar & zip).

@sumerjabri sumerjabri changed the title from [CrafterCms] Bundle tool should allow users to choose what kind of archive they want to use. to [craftercms] Bundle tool should allow users to choose what kind of archive they want to use. Dec 4, 2017

@sumerjabri sumerjabri added this to the Crafter CMS v3.0.3 milestone Dec 4, 2017

@sumerjabri sumerjabri added this to Backlog in Crafter CMS v3.0.x via automation Dec 4, 2017

@cortiz

This comment has been minimized.

Show comment
Hide comment
@cortiz

cortiz Dec 15, 2017

Member

Partially solve in #1708
using delivery_bundle_tar, delivery_bundle_zip ,authoring_bundle_tar or authoring_bundle_zip
@sumerjabri do we still want to do crafter.bundle.archive=tar|zip ?

Member

cortiz commented Dec 15, 2017

Partially solve in #1708
using delivery_bundle_tar, delivery_bundle_zip ,authoring_bundle_tar or authoring_bundle_zip
@sumerjabri do we still want to do crafter.bundle.archive=tar|zip ?

sumerjabri added a commit that referenced this issue Dec 18, 2017

@sumerjabri

This comment has been minimized.

Show comment
Hide comment
@sumerjabri

sumerjabri Dec 18, 2017

Member

@cortiz @alhambrav let's document it as you have it and decide afterward.

Member

sumerjabri commented Dec 18, 2017

@cortiz @alhambrav let's document it as you have it and decide afterward.

@cortiz

This comment has been minimized.

Show comment
Hide comment
@cortiz

cortiz Dec 18, 2017

Member

Param Added in PR #1711

Member

cortiz commented Dec 18, 2017

Param Added in PR #1711

@sumerjabri

This comment has been minimized.

Show comment
Hide comment
@sumerjabri

sumerjabri Dec 18, 2017

Member

@alhambrav please validate and close if done.

Member

sumerjabri commented Dec 18, 2017

@alhambrav please validate and close if done.

@alhambrav

This comment has been minimized.

Show comment
Hide comment
@alhambrav

alhambrav Dec 19, 2017

Member

Verified.

Member

alhambrav commented Dec 19, 2017

Verified.

@alhambrav alhambrav closed this Dec 19, 2017

Crafter CMS v3.0.x automation moved this from Backlog to Completed Dec 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment