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

Auto-detect collections, so it's just plug-and-play with no configuration #9

Closed
JackAdams opened this issue Feb 20, 2015 · 8 comments
Closed

Comments

@JackAdams
Copy link
Contributor

No need for the config setup step - just make it plug-and-play, using a default config. Devs can put in their own custom config if they want to.

Use the (very lightweight) (dburles:mongo-collection-instances package.

By default you get all the collections (including namespaced ones) in Mongol. If someone only wants a subset of the collections, they can Session.set("Mongol", {collections: [ ... ], ... }) as usual.

Happy to put a PR together once there's a repo I can fork.

@msavin
Copy link
Owner

msavin commented Feb 24, 2015

Jack,

That's a great idea. With MongoInspector, we had some issues with this approach when the package wasn't listed at the top of the packages list. If you think this approach will have the same issues, maybe we should preserve the option for manual input.

I just pushed up the package source code, looking forward to seeing your PR!

  • Max

@JackAdams
Copy link
Contributor Author

Yeah, man. Putting that PR together is the first thing I'll be doing when I get home from work tonight. I'm really excited about Mongol and looking forward to using it in all my projects. "Insanely useful" is an understatement.

@msavin
Copy link
Owner

msavin commented Feb 24, 2015

Awesome man! I can't wait to see it. Let me know if you need any help along the way.

@msavin
Copy link
Owner

msavin commented Feb 24, 2015

On a side note; we did not use the same exact code as @dburles so maybe it won't have the same issue

@SantoshSrinivas79
Copy link

Just wondering ... is the auto detection incorporated? or was this just closed out.

@msavin
Copy link
Owner

msavin commented Feb 25, 2015

Implemented in 0.4.0 - just run meteor update

Sent from my iPhone

On Feb 25, 2015, at 1:01 PM, fountainhead notifications@github.com wrote:

Just wondering ... is the auto detection incorporated? or was this just closed out.


Reply to this email directly or view it on GitHub.

@SantoshSrinivas79
Copy link

What is the configuration i need to do? I currently have the below code in a client .js file

Meteor.startup(function() {

  Session.set("Mongol", {
    'display': true,
    'opacity_normal': ".7",
    'opacity_expand': ".9"
  });

});

@msavin
Copy link
Owner

msavin commented Feb 25, 2015

Just remove it 😄

Sent from my iPhone

On Feb 25, 2015, at 1:21 PM, fountainhead notifications@github.com wrote:

What is the configuration i need to do? I currently have the below code in a client .js file

Meteor.startup(function() {

Session.set("Mongol", {
'display': true,
'opacity_normal': ".7",
'opacity_expand': ".9"
});

});

Reply to this email directly or view it on GitHub.

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