Page MenuHomePhabricator
Differential D12082

Explicitly draw progress bar when resuming file uploads in `arc upload`
ClosedPublic
Actions

Authored by epriestley on Mar 14 2015, 4:35 PM.
Tags
None
Referenced Files
F15179859: D12082.id29079.diff
Wed, Feb 19, 4:47 PM
Unknown Object (File)
Fri, Feb 14, 2:12 PM
Unknown Object (File)
Mon, Feb 10, 3:35 AM
Unknown Object (File)
Sat, Feb 8, 3:04 AM
Unknown Object (File)
Sat, Feb 8, 3:03 AM
Unknown Object (File)
Sat, Feb 8, 3:03 AM
Unknown Object (File)
Tue, Feb 4, 5:35 AM
Unknown Object (File)
Tue, Feb 4, 3:42 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
chad
Maniphest Tasks
T7149: Allow users to import data into a new Phacility instance
Commits
rARCb961869edac9: Explicitly draw progress bar when resuming file uploads in `arc upload`
Summary

Ref T7149. Make sure we sit at "Resuming: 60%" or whatever while uploading the first chunk.

Test Plan

Ran arc upload on a large file, cancelled it, resumed it, got sensible progress bar.

Diff Detail

Repository
rARC Arcanist
Branch
chunk3
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4888
Build 4906: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 29064.Mar 14 2015, 4:35 PM
epriestley retitled this revision from to Explicitly draw progress bar when resuming file uploads in `arc upload`.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T7149: Allow users to import data into a new Phacility instance.
Herald added a subscriber: epriestley. · View Herald TranscriptMar 14 2015, 4:35 PM
chad accepted this revision.Mar 14 2015, 4:38 PM
chad added a reviewer: chad.
This revision is now accepted and ready to land.Mar 14 2015, 4:38 PM
btrahan accepted this revision.Mar 15 2015, 6:09 PM
btrahan edited edge metadata.
Closed by commit rARCb961869edac9: Explicitly draw progress bar when resuming file uploads in `arc upload` (authored by epriestley, committed by epriestley). · Explain WhyMar 15 2015, 6:32 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
workflow/
2 lines

Diff 29064

View Options

src/workflow/ArcanistUploadWorkflow.php

Loading...
Phabricator · Built by Phacility