-
Notifications
You must be signed in to change notification settings - Fork 873
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 getBytes() #5672
Add getBytes() #5672
Conversation
🦋 Changeset detectedLatest commit: d119bfb The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
…sdk into mrschmidt/getbytes
Binary Size ReportAffected SDKs
Test Logs
|
Size Analysis ReportAffected Products
|
* Cloud Storage bucket. See also | ||
* https://cloud.google.com/storage/docs/configuring-cors | ||
* | ||
* @public |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@egilmorez can you please scan the PR and review the comments with the @public
tag? Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was hoping @markarndt could give that a go . . . but if he's too busy and there's urgency, I can take a look this afternoon for sure.
Co-authored-by: Feiyang <feiyangc@google.com>
Co-authored-by: Feiyang <feiyangc@google.com>
Co-authored-by: Feiyang <feiyangc@google.com>
Co-authored-by: Feiyang <feiyangc@google.com>
…sdk into mrschmidt/getbytes
@@ -100,6 +100,27 @@ describeSpec('Queries:', [], () => { | |||
}); | |||
}); | |||
|
|||
Caused by: java.lang.AssertionError: expected:< |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This .... ehmm... seems unrelated. Removed.
Co-authored-by: Feiyang <feiyangc@google.com>
…sdk into mrschmidt/getbytes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cleaned up the PR and simplified some of the logic. It should now be ready for another round.
@@ -100,6 +100,27 @@ describeSpec('Queries:', [], () => { | |||
}); | |||
}); | |||
|
|||
Caused by: java.lang.AssertionError: expected:< |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This .... ehmm... seems unrelated. Removed.
…sdk into mrschmidt/getbytes
packages/storage/src/api.ts
Outdated
* https://cloud.google.com/storage/docs/configuring-cors | ||
* | ||
* @public | ||
* @param ref - StorageReference where data should be download. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"..should be downloaded."
Adds getBytes, getBlob and getStream APIs which allows file downloads without a public download URL. These bytes can then be used directly as an image source.
This PR contains a bunch of refactors to make this possible:
Connection<string>
,Connection<ArrayBuffer>
andConnection<ReadableStream>
.Fixes #76