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

Fix multer options #256

Merged
merged 2 commits into from
Nov 16, 2016
Merged

Fix multer options #256

merged 2 commits into from
Nov 16, 2016

Conversation

kvcpr
Copy link

@kvcpr kvcpr commented Nov 16, 2016

There is no "inMemory" options for multer.

There is no "inMemory" options.
const multer = require('multer')({
inMemory: true,
const multer = Multer({
storage: Multer.MemoryStorage,
Copy link
Member

Choose a reason for hiding this comment

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

From multer's readme it seems like this should be storage: Multer.memoryStorage() right?

Copy link
Author

Choose a reason for hiding this comment

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

Yeah it's. Sorry my bad. Changed

@jmdobry jmdobry merged commit eeeaa98 into GoogleCloudPlatform:master Nov 16, 2016
@jmdobry
Copy link
Member

jmdobry commented Nov 16, 2016

Thanks!

grayside pushed a commit that referenced this pull request Oct 26, 2022
* updated CHANGELOG.md [ci skip]

* updated package.json [ci skip]

* updated samples/package.json [ci skip]
grayside pushed a commit that referenced this pull request Nov 3, 2022
* updated CHANGELOG.md [ci skip]

* updated package.json [ci skip]

* updated samples/package.json [ci skip]
ahrarmonsur pushed a commit that referenced this pull request Nov 8, 2022
* updated CHANGELOG.md

* updated package.json

* updated samples/package.json
ace-n pushed a commit that referenced this pull request Nov 10, 2022
* updated CHANGELOG.md

* updated package.json

* updated samples/package.json
ahrarmonsur pushed a commit that referenced this pull request Nov 18, 2022
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
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