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

Meteor update #2210

Merged
merged 3 commits into from
Mar 6, 2017
Merged

Meteor update #2210

merged 3 commits into from
Mar 6, 2017

Conversation

brylie
Copy link
Contributor

@brylie brylie commented Mar 1, 2017

Update Meteor core and most dependencies.

@marla-singer
Copy link
Contributor

@brylie Meteor can't start on my side
ShareJS: using unsupported db type none, falling back to in-memory.
and

TypeError: Cannot read property 'call' of undefined
at Restivus.<anonymous> (packages/nimble_restivus/lib/restivus.coffee:126:18)
...
at meteorInstall.core.server.api.js (core/server/api.js:44:9)
...
at meteorInstall.apis.server.api.js (apis/server/api.js:1:88)
...

.meteor/versions Outdated
minifier-css@1.2.15
minifier-js@1.2.15
minimongo@1.0.19
mizzao:sharejs@0.9.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's pin this package

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, cool.

.meteor/versions Outdated
mongo-id@1.0.6
mongo-livedata@1.0.12
nimble:restivus@0.8.11
nimble:restivus@0.8.12
Copy link
Contributor

Choose a reason for hiding this comment

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

And this one

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I am getting a related console error:

TypeError: Cannot read property 'call' of undefined
     at Restivus.<anonymous> (packages/nimble_restivus/lib/restivus.coffee:126:18)
     at Array.forEach (native)
     at Function._.each._.forEach (packages/underscore.js:139:11)
     at Restivus.Assets.Restivus.Restivus.addCollection (packages/nimble_restivus/lib/restivus.coffee:120:9)
     at meteorInstall.core.server.api.js (core/server/api.js:44:9)

@marla-singer marla-singer self-assigned this Mar 3, 2017
@brylie
Copy link
Contributor Author

brylie commented Mar 6, 2017

@marla-singer I pinned nimble:restivus and mizzao:sharejs. Please review.

@marla-singer
Copy link
Contributor

@brylie Works fine. It will be really good to have system testing for checking PR like this one ☂️

@55
Copy link
Contributor

55 commented Mar 6, 2017

I also want to test it before merging. Will do it now and let you know.

@marla-singer marla-singer added this to the Sprint 38 milestone Mar 6, 2017
@55
Copy link
Contributor

55 commented Mar 6, 2017

@brylie @marla-singer everything works as expected to me.

@marla-singer
Copy link
Contributor

@NNN @brylie Nice, merge it

@marla-singer marla-singer modified the milestones: Sprint 37, Sprint 38 Mar 6, 2017
@marla-singer marla-singer merged commit c054d76 into develop Mar 6, 2017
@marla-singer marla-singer deleted the feature/meteor-update branch March 6, 2017 13:33
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.

None yet

3 participants