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.

Sign up for GitHub

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

Jump to bottom

Better file uploads APIs with cleaner UX and upload progress #110

Merged
merged 19 commits into from
Feb 24, 2018
Merged

Better file uploads APIs with cleaner UX and upload progress #110

merged 19 commits into from
Feb 24, 2018

Conversation

kabirbaidhya
Copy link
Owner

@kabirbaidhya kabirbaidhya commented Jan 2, 2018 •
edited
Loading

  • Better file upload/transfer API with upload progress. Part of Fabric Independence - rewrite core SSH and remote functionality directly over paramiko #99
  • Refactor and move upload_dir function to transfers module
  • Update tests for upload_dir()
  • Formatting default upload progress output
  • Make use of transfers.upload_dir() in node deployment
  • Fix bulk upload issues
  • Rename Uploader to DirectoryUploader
  • Extract upload status types to a separte constants file
  • Introduce FileUploader class
  • Implement transfers.upload() function based on FileUploader
  • Introduce Uploader base class and refactor code
  • Extract status constants and refactor code
  • Better variable naming for FileUploader
  • Introduce BulkUploader
  • Improvements on bulk uploads
  • Move tmp_path to boss.core.fs and fix it's non-unique path returning issue
  • More tests

@kabirbaidhya kabirbaidhya self-assigned this Jan 2, 2018
@kabirbaidhya kabirbaidhya changed the base branch from master to dev February 6, 2018 19:38
@kabirbaidhya
Implement Uploader class for smoother directory uploads
14366a9
@kabirbaidhya
Refactor and move upload_dir function to transfers module
e51d1dd
@kabirbaidhya
Update tests for upload_dir()
c32ce32
@kabirbaidhya
Formatting default upload progress output
23b3936
@kabirbaidhya
Make use of transfers.upload_dir() in node deployment
9ae2fb3
@kabirbaidhya
Rename Uploader to DirectoryUploader
398606c
@kabirbaidhya
Extract upload status types to a separte constants file
f084831
@kabirbaidhya
Extract status constants and refactor code
bcebf34
@kabirbaidhya
Introduce FileUploader class
fc12c3e
@kabirbaidhya
Implement transfers.upload() function based on FileUploader
2466592
@kabirbaidhya
Better variable naming for FileUploader
c130540
@kabirbaidhya
Integration tests for transfers.upload()
11b4f02
@kabirbaidhya
Introduce Uploader base class and refactor code
355cfe8
@kabirbaidhya
Introduce BulkUploader
ba50391
@kabirbaidhya
Improvements on bulk uploads
1588555
@kabirbaidhya
Fix directory upload tests
81d9cc6
@kabirbaidhya
Fix tmp_path() function issue and extract it to fs module
8397310
@kabirbaidhya kabirbaidhya removed the wip label Feb 24, 2018
@kabirbaidhya kabirbaidhya mentioned this pull request Feb 24, 2018
Open
12 tasks
@kabirbaidhya kabirbaidhya merged commit 7918341 into dev Feb 24, 2018
@kabirbaidhya kabirbaidhya deleted the uploads branch February 24, 2018 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@kabirbaidhya kabirbaidhya

Labels
deployment experimental interesting internals remote execution ux
Projects
None yet
Milestone
First stable release v1.0.0
Development

Successfully merging this pull request may close these issues.
1 participant
@kabirbaidhya