Skip to content

TT-7181 Integrate JSZip to verify if itf project can be imported#297

Merged
sarahentzel merged 1 commit intodevelopfrom
TT-7181-itf
May 6, 2026
Merged

TT-7181 Integrate JSZip to verify if itf project can be imported#297
sarahentzel merged 1 commit intodevelopfrom
TT-7181-itf

Conversation

@gtryus
Copy link
Copy Markdown
Contributor

@gtryus gtryus commented May 6, 2026

  • Added jszip dependency to manage ITF files.
  • Implemented readItfEmbeddedProject utility to extract project data from ITF files.
  • Enhanced uploadITF function to validate embedded project references before file upload, improving error handling and user feedback.

- Added `jszip` dependency to manage ITF files.
- Implemented `readItfEmbeddedProject` utility to extract project data from ITF files.
- Enhanced `uploadITF` function to validate embedded project references before file upload, improving error handling and user feedback.
Copy link
Copy Markdown
Collaborator

@sarahentzel sarahentzel left a comment

Choose a reason for hiding this comment

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

We already import adm-zip and node-stream-zip. Can we use one of those?

@gtryus
Copy link
Copy Markdown
Contributor Author

gtryus commented May 6, 2026

We already import adm-zip and node-stream-zip. Can we use one of those?

these two dependencies only work from the desktop app. I think jszip is supposed to work from the web.

@gtryus
Copy link
Copy Markdown
Contributor Author

gtryus commented May 6, 2026

@sarahentzel as an alternative, the test could be run in the cloud but then the user wouldn't know until after we tried to upload the itf.

@sarahentzel sarahentzel merged commit 7210c33 into develop May 6, 2026
2 checks passed
@sarahentzel sarahentzel deleted the TT-7181-itf branch May 6, 2026 16:40
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.

2 participants