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

Conflict with aldeed:tabular? #133

Open
brylie opened this issue Jul 2, 2015 · 4 comments
Open

Conflict with aldeed:tabular? #133

brylie opened this issue Jul 2, 2015 · 4 comments

Comments

@brylie
Copy link

brylie commented Jul 2, 2015

When installing this package on a project that uses aldeed:tabular, I get the following error:

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:100:27
     at Function._.each._.forEach (packages/underscore/underscore.js:113:1)
     at adminCreateTables (packages/yogiben:admin/lib/both/startup.coffee:77:4)
     at __coffeescriptShare (packages/yogiben:admin/lib/both/startup.coffee:178:2)
     at /home/brylie/Code/Apinf/api-umbrella-dashboard/.meteor/local/build/programs/server/boot.js:229:5

I am not sure where to troubleshoot this issue, e.g. with yogiben:admin or aldeed:tabular. However, removing the collection hooks package resolves the errors.

@brylie
Copy link
Author

brylie commented Jul 6, 2015

There is one solution posted on the yogiben:meteor-admin #216. It seems that editing the .meteor/packages file and moving matb33:collection-hooks directly below aldeed:collection2 resolves the issue. The underlying cause is still under question.

@brylie
Copy link
Author

brylie commented Jul 6, 2015

I have posted an issue describing the load order of the 'comments' and 'collection-hooks' packages in yogiben:meteor-starter that may be the underlying cause of this issue.

@matb33
Copy link
Collaborator

matb33 commented Jul 6, 2015

At some point I will look into all this and find a definitive solution

#125 (comment)

@StorytellerCZ
Copy link
Member

@matb33 Do you think this is now solved?

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

No branches or pull requests

3 participants