Skip to content

Conversation

@ZgjimHaziri
Copy link
Contributor

@ZgjimHaziri ZgjimHaziri commented Aug 12, 2025

Description

Enable exporting files as unzipped and importing unzipped directories.

  • Added a --unzip option in export which decides if the result of the download is a zip or a directory.
  • In import we added a --directory option as an alternative to --file.
    • Added validation for the different combinations of the import options.

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

@ZgjimHaziri ZgjimHaziri requested a review from a team as a code owner August 12, 2025 13:03
promeris
promeris previously approved these changes Aug 12, 2025
Buqeta
Buqeta previously approved these changes Aug 12, 2025
@ZgjimHaziri ZgjimHaziri dismissed stale reviews from Buqeta and promeris via 417e0c3 August 12, 2025 14:44
Buqeta
Buqeta previously approved these changes Aug 13, 2025
promeris
promeris previously approved these changes Aug 13, 2025
jetakasabaqi
jetakasabaqi previously approved these changes Aug 13, 2025
@ZgjimHaziri ZgjimHaziri dismissed stale reviews from jetakasabaqi, promeris, and Buqeta via 9278bc6 August 13, 2025 13:41
@ZgjimHaziri
Copy link
Contributor Author

Changed the implementation by adding a --directory option for folders instead of using the --file one for both. It seemed more fit from a UX perspective because of the naming of --file.

jetakasabaqi
jetakasabaqi previously approved these changes Aug 13, 2025
LaberionAjvazi
LaberionAjvazi previously approved these changes Aug 13, 2025
Buqeta
Buqeta previously approved these changes Aug 14, 2025
promeris
promeris previously approved these changes Aug 14, 2025
ksalihu
ksalihu previously approved these changes Aug 14, 2025
@ZgjimHaziri ZgjimHaziri merged commit 2ce2217 into master Aug 14, 2025
1 check passed
@ZgjimHaziri ZgjimHaziri deleted the TA-4103-zip-unzip branch August 14, 2025 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

7 participants