Skip to content
This repository has been archived by the owner on Mar 14, 2019. It is now read-only.

Fix gridFS compatibility with Meteor 1.6 #1018

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Herteby
Copy link
Contributor

@Herteby Herteby commented Nov 6, 2017

Update mongodb to 2.2.33

Update mongodb to 2.2.33
@Herteby
Copy link
Contributor Author

Herteby commented Nov 6, 2017

After 1.6, my Meteor build started failing with { Error: Cannot find module '../build/Release/bson'

When I googled the error I found this comment by @badmark : meteor/meteor#7477 (comment)
So I figured that it was the same thing again, and the mongodb dependency had to be updated to the same version that Meteor 1.6 uses. That fix solved the problem for me.

@Herteby Herteby changed the title Fix compatibility with Meteor 1.6 Fix gridFS compatibility with Meteor 1.6 Nov 6, 2017
Copy link
Contributor

@nooitaf nooitaf left a comment

Choose a reason for hiding this comment

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

thanks, always good to go with the times :)

@mattgibson81
Copy link

Hi,

I have followed these instructions and this has resolved my issue with the error ({ Error: Cannot find module '../build/Release/bson') but now my upload function has broken.

When I attempt to upload I get the error...

The provided value 'undefined' is not a valid enum value of type XMLHttpRequestResponseType.
httpCall @ cfs_upload-http.js?hash=de2b2d82cc6e3296c9e82723ace0812df1773294:140

I really need this to work as I have many sites using these packages with documents uploaded already and I don't want these to stop working.

Any help would be much appreciated. If you need any further information please let me know.

Thanks, Matt.

@harryadel
Copy link
Member

harryadel commented Jan 7, 2019

Hi @Herteby try to pull the latest changes from master and Travis issue'll be gone this time leaving only the package.js conflict which I hope you can address in a timely manner, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants