Skip to content

feat: Add "Compress to zip" action with declarative UI#358

Merged
juliusknorr merged 3 commits intomainfrom
add-compress-to-zip-action-with-declarative-ui
Jan 23, 2026
Merged

feat: Add "Compress to zip" action with declarative UI#358
juliusknorr merged 3 commits intomainfrom
add-compress-to-zip-action-with-declarative-ui

Conversation

@danxuliu
Copy link
Member

@danxuliu danxuliu commented Sep 16, 2025

Fixes #347

A new endpoint was added to compress a given path to zip. A path had to be used instead of a file ID as the same file ID could belong to different paths, so just from the file ID it would not be possible to generate the path to the target zip file.

A unique path will be generated for the target zip file when the zip job is created. However, note that creating the zip file will anyway fail if the path exists once the job is executed.

@danxuliu danxuliu added enhancement New feature or request 2. developing labels Sep 16, 2025
@danxuliu danxuliu force-pushed the add-compress-to-zip-action-with-declarative-ui branch from e5bd321 to 4e2b1cf Compare September 16, 2025 13:58
@danxuliu danxuliu force-pushed the add-compress-to-zip-action-with-declarative-ui branch from 8f0e3ac to 7484c2a Compare October 7, 2025 17:45
@danxuliu danxuliu force-pushed the add-compress-to-zip-action-with-declarative-ui branch 2 times, most recently from 5117f32 to 40652f6 Compare January 5, 2026 12:13
@danxuliu
Copy link
Member Author

danxuliu commented Jan 5, 2026

Rebased and fixed up. It should be ready for reviewing.

CI failures for ZipFeatureTest::testZip PHP unit test are unrelated, and they also happen in main branch.

@danxuliu danxuliu marked this pull request as ready for review January 5, 2026 12:14
@tobiasKaminsky
Copy link
Member

@juliusknorr please schedule this for NC33

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
A new endpoint was added to compress a given path to zip. A path had to
be used instead of a file ID as the same file ID could belong to
different paths, so just from the file ID it would not be possible to
generate the path to the target zip file.

A unique path will be generated for the target zip file when the zip job
is created. However, note that creating the zip file will anyway fail if
the path exists once the job is executed.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@juliusknorr juliusknorr force-pushed the add-compress-to-zip-action-with-declarative-ui branch from 40652f6 to ccd7238 Compare January 23, 2026 11:27
Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

Code looks good and CI failures are unrelated

@juliusknorr juliusknorr merged commit 05eb350 into main Jan 23, 2026
28 of 45 checks passed
@juliusknorr juliusknorr deleted the add-compress-to-zip-action-with-declarative-ui branch January 23, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Declarative UI: use a file/folder to create a zip file

3 participants