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

56 tar downloads #82

Merged
merged 19 commits into from
Oct 25, 2023
Merged

56 tar downloads #82

merged 19 commits into from
Oct 25, 2023

Conversation

ri-pandey
Copy link
Contributor

@ri-pandey ri-pandey commented Oct 10, 2023

Description

Added ability to download datasets' tar files (that are staged on Slate-Scratch) from the browser.

Related Issue(s)

Closes #56

Changes Made

  • Feature added
  • Documentation updated
  • Other changes:
  • Changed the location previously being used to stage the tar file
  • Created a symlink on Slate-Scratch that points to the location to the staged tar file
  • Added a new column to 'dataset' table (bundle_size) which tracks the size of the staged tar file
  • Added tar file deletion to the purge_staged_datasets.py script
  • Added a third option to DatasetDownloadModal.vue, which initiates the tar file download. Tweaked wording.
  • Updated existing HTTP endpoint to support both file as well as dataset downloads.
  • Updates statistics code accordingly.
  • Tweaked existing comments from a previous ticket, and chart colors for a certain chart.

Checklist

Before submitting this PR, please make sure that:

  • Your code passes linting and coding style checks.
  • Documentation has been updated to reflect the changes.
  • You have requested a review from at least one team member.
  • Any relevant issue(s) have been linked to this PR.

Additional Information

Add any other information or context that may be relevant to this PR. This can include potential impacts, known issues, or future work related to this change.

@ri-pandey ri-pandey force-pushed the 56-tar-downloads branch 2 times, most recently from b83b3d5 to 962dd02 Compare October 12, 2023 13:37
@deepakduggirala deepakduggirala force-pushed the 56-tar-downloads branch 2 times, most recently from a7b8176 to 6cf437d Compare October 25, 2023 20:25
@deepakduggirala deepakduggirala merged commit d922184 into main Oct 25, 2023
@deepakduggirala deepakduggirala deleted the 56-tar-downloads branch October 25, 2023 20:36
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