Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Support StorageReference#writeToFile #512

Merged
merged 2 commits into from
May 1, 2018

Conversation

szakarias
Copy link
Contributor

No description provided.

@szakarias szakarias force-pushed the storageReferenceWriteToFile branch 5 times, most recently from 605efe4 to 0335232 Compare April 30, 2018 18:09
@szakarias szakarias force-pushed the storageReferenceWriteToFile branch from 0335232 to 74e8d34 Compare April 30, 2018 18:18
@szakarias szakarias requested a review from mravn-google April 30, 2018 18:19
Copy link
Contributor

@mravn-google mravn-google left a comment

Choose a reason for hiding this comment

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

LGTM

await tempFile.create();
assert(await tempFile.readAsString() == "");
final StorageFileDownloadTask task = ref.writeToFile(tempFile);
final int bytes = (await task.future).totalByteCount;
Copy link
Contributor

Choose a reason for hiding this comment

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

[nit] Maybe rename to byteCount so we don't confuse it with the actual bytes downloaded.

@szakarias szakarias merged commit c5678eb into flutter:master May 1, 2018
@szakarias szakarias deleted the storageReferenceWriteToFile branch May 1, 2018 07:15
haydenflinner pushed a commit to haydenflinner/plugins that referenced this pull request May 4, 2018
slightfoot pushed a commit to slightfoot/plugins that referenced this pull request Jun 5, 2018
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants