search

Meteor-Community-Packages / meteor-collection-hooks

Meteor Collection Hooks
https://atmospherejs.com/matb33/collection-hooks
MIT License
657 stars 92 forks source link

Incompatible with meteor-starter #125

Closed femiveys closed 9 years ago

femiveys commented 9 years ago

Steps to reproduce:

Then I get this error: Your app is crashing. Here's the latest log.

Meteor._wrapAsync has been renamed to Meteor.wrapAsync

/home/femi/.meteor/packages/meteor-tool/.1.1.3.4sddkj++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/nodemodules/fibers/future.js:245 throw(ex); ^ Error: Tabular.Table options must specify collection at new Tabular.Table (packages/aldeed:tabular/common.js:20:1) at packages/yogiben:admin/lib/both/startup.coffee:89:27 at Function..each._.forEach (packages/underscore/underscore.js:113:1) at adminCreateTables (packages/yogiben:admin/lib/both/startup.coffee:66:4) at __coffeescriptShare (packages/yogiben:admin/lib/both/startup.coffee:114:2) at /home/femi/Dropbox/IT Simply Works/Klanten/www/a/.meteor/local/build/programs/server/boot.js:229:5 Exited with code: 8 Your application is crashing. Waiting for file change.

matb33 commented 9 years ago

Did you try with: meteor add matb33:collection-hooks? I'm actually not sure what will get loaded with meteor add meteor-collection-hooks to be honest...

femiveys commented 9 years ago

This gives me exactly the same error.

rclai commented 9 years ago

Are you on an old browser?

femiveys commented 9 years ago

No, latest chrome. But the error also appears several times in the console.

matb33 commented 9 years ago

You'll need to edit .meteor/packages and move the reference for matb33:collection-hooks further up the top, as long as it's above the comments package. We haven't figured out a way around this issue, and is actually a main driver towards implementing collection hooks into core (or having a core collection extension API).

femiveys commented 9 years ago

I can confirm this works perfectly. Thanks a lot. The actual goal was socialize:friendships which depends on socialize:base-model, which depends on matb33:collection-hooks. With this workaround, this works as well.