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

Get all collections with instances pkg #251

Closed
matteodem opened this issue Nov 14, 2014 · 8 comments
Closed

Get all collections with instances pkg #251

matteodem opened this issue Nov 14, 2014 · 8 comments
Assignees
Labels

Comments

@matteodem
Copy link
Contributor

@matteodem matteodem commented Nov 14, 2014

Using https://github.com/dburles/mongo-collection-instances could help with the mess of searching for all collections on the global scope and by accessing the mongo driver. The simple approach by this package could be of a lot of help, what do you guys think? I'd start with it soonish.

see https://github.com/gterrono/houston/blob/master/server/publications.coffee#L45

@matteodem matteodem self-assigned this Nov 14, 2014
@AlexeyMK
Copy link
Collaborator

@AlexeyMK AlexeyMK commented Nov 14, 2014

@gterrono and I were actually talking about that this morning. There's some concern whether this approach would capture some of the earliest collections that get created in other package code, before the wrapper is in place. Definitely worth a try, though. I think @gterrono was thinking of starting with that?

@gterrono
Copy link
Owner

@gterrono gterrono commented Nov 14, 2014

I think I'll put off doing that for now, so definitely go for it @matteodem

@matteodem
Copy link
Contributor Author

@matteodem matteodem commented Nov 15, 2014

can't start, also getting those '_houston_undefined_insert' method is already defined errors, when doing following:

meteor create --example todos && cd todos
meteor add houston:admin
@gterrono
Copy link
Owner

@gterrono gterrono commented Nov 15, 2014

I think your app is probably not using the most up to date version of houston. Check in .meteor/versions

@gterrono
Copy link
Owner

@gterrono gterrono commented Nov 15, 2014

oh, it looks like the todo example app relies on iron:router at 0.9.4, and which houston 1.2.0 and up doesn't support. You'll need to use an app that allows iron:router 1.0.0+.

@gterrono
Copy link
Owner

@gterrono gterrono commented Nov 15, 2014

The test app is a good one to use to develop houston because it's already set up to symlink back to the package. It's in test/test_app I believe.

@matteodem
Copy link
Contributor Author

@matteodem matteodem commented Nov 15, 2014

oh okay, perfect! thanks

@matteodem
Copy link
Contributor Author

@matteodem matteodem commented Nov 21, 2014

this is done and in the master :)

@matteodem matteodem closed this Nov 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.