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

Add option to download all artifacts on the registry webapp #107

Closed
sripwoud opened this issue Jun 24, 2024 · 2 comments · Fixed by #108
Closed

Add option to download all artifacts on the registry webapp #107

sripwoud opened this issue Jun 24, 2024 · 2 comments · Fixed by #108
Assignees
Labels
enhancement New feature or request

Comments

@sripwoud
Copy link
Member

sripwoud commented Jun 24, 2024

No description provided.

@sripwoud
Copy link
Member Author

We may consider downloading directly form the npm registry there as it can provide archives containing all files.
Otherwise we will have to perform all these steps client side:

  • download blobs all artifacts in parallel in memory
  • zip everything
  • effectively download the archive

@cedoor cedoor changed the title web-app: add option to download all artifacts Add option to download all artifacts on the registry webapp Jun 26, 2024
@cedoor
Copy link
Member

cedoor commented Jun 26, 2024

We may consider downloading directly form the npm registry there as it can provide archives containing all files.

Right, using NPM sounds smarter.

@cedoor cedoor added the enhancement New feature or request label Jun 26, 2024
@cedoor cedoor moved this from 📋 Backlog to 🏗 In Progress in SNARK Artifacts Jun 26, 2024
@cedoor cedoor moved this from 🏗 In Progress to 👀 In Review in SNARK Artifacts Jun 26, 2024
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✔️ Done in SNARK Artifacts Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: ✔️ Done
Development

Successfully merging a pull request may close this issue.

2 participants