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

be able to pass userId to addFile (API) #305

Merged
merged 1 commit into from
Dec 14, 2016

Conversation

menelike
Copy link
Contributor

@menelike menelike commented Dec 12, 2016

solves #298

Do we need to to set userId to null? Isn't it better to skip that (userId === undefined)?

no expert in CoffeeScript, review advised

@dr-dimitru
Copy link
Member

@menelike the logic behind current implementation is simple - always return null if there is no user

Well, if we are going to let set an userId here, should we add same ability to other methods, for consistency?

@dr-dimitru dr-dimitru merged commit 7003754 into veliovgroup:dev Dec 14, 2016
@menelike
Copy link
Contributor Author

@dr-dimitru sorry I missed that one, hope it worked out fine

@dr-dimitru
Copy link
Member

Yes, already pushed to dev

dr-dimitru added a commit that referenced this pull request Dec 15, 2016
 - NPM dependencies update
 - Support for updated *Buffer* API on node >= 6.*
 - Methods `addFile`, `write` and `load` now supports `userId` option,
see #305
 - Fix for unfinished uploads, now files is removed by upload TTL. See
#267, thanks to @ck23onGithub
 - Better FS-stream management on server. Less memory and CPU
consumption. Only one descriptor per upload. Fix FS-permissions
 - `.unlink()` method now supports callback
 - Docs for [MUP/Docker
integration](https://github.com/VeliovGroup/Meteor-Files/wiki/MeteorUp-(
MUP)-Usage)
 - Fix #293
 - Fix #299
 - BEtter AWS dos, see #282
 - `storagePath` now always is function, if *String* is passed - it
will be wrapped into function
 - Please participate at [this
thread](#309) to give
a right path for Meteor-Files v2.0
@dr-dimitru dr-dimitru mentioned this pull request Dec 15, 2016
dr-dimitru added a commit that referenced this pull request Dec 15, 2016
v1.7.6
 - NPM dependencies update
 - Support for updated *Buffer* API on node >= 6.*
 - Methods `addFile`, `write` and `load` now supports `userId` option, see #305
 - Fix for unfinished uploads, now files is removed by upload TTL. See #267, thanks to @ck23onGithub
 - Better FS-stream management on server. Less memory and CPU consumption. Only one descriptor per upload. Fix FS-permissions
 - `.unlink()` method now supports callback
 - Docs for [MUP/Docker integration](https://github.com/VeliovGroup/Meteor-Files/wiki/MeteorUp-(MUP)-Usage)
 - Fix #293
 - Fix #299
 - BEtter AWS dos, see #282
 - `storagePath` now always is function, if *String* is passed - it will be wrapped into function
 - Please participate at [this thread](#309) to give a right path for Meteor-Files v2.0
@dr-dimitru
Copy link
Member

Published in v1.7.6

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