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

Update upload_server.js #29

Closed
wants to merge 1 commit into from
Closed

Conversation

msj121
Copy link
Contributor

@msj121 msj121 commented May 3, 2016

I think this will fix tomitrescak/meteor-uploads#220

@rvetere
Copy link

rvetere commented Jul 10, 2016

please merge this!

@tomitrescak
Copy link
Owner

I've done it differently, but resolves the same problem. New version was published.

@rvetere
Copy link

rvetere commented Jul 11, 2016

Awesome, thanks a lot! 👍

@msj121
Copy link
Contributor Author

msj121 commented Oct 28, 2016

@tomitrescak This is still not fixed, I have updated to the new version and the same fix works as far as I can tell...

Please see #34 for confirmation that the bug persists....

It would be nice to confirm that this won't ruin the package for those without subfolders though

@msj121
Copy link
Contributor Author

msj121 commented Oct 28, 2016

There actually was a typo, I have that fixed in PR #2.

@msj121
Copy link
Contributor Author

msj121 commented Oct 28, 2016

@tomitrescak Please add me as a code collaborator if this is going to be deprecated as well. Thanks.

@tomitrescak
Copy link
Owner

I just did. All yours ;) thanks for taking over.

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.

Sudden issue with uploaded files & their directory
3 participants