Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Debug mongoose queries and Add angular-translate #143

Closed
wants to merge 5 commits into from

Conversation

rubenjgarcia
Copy link

Option to debug mongoose queries on development environment

Schema = mongoose.Schema;

mongoose.set('debug', config.mongoose ? config.mongoose.debug || false : false);
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this could be more simply written as:

mongoose.set('debug', config.mongoose && config.mongoose.debug);

Copy link
Author

Choose a reason for hiding this comment

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

I've changed it. Thanks!!!

@rubenjgarcia rubenjgarcia reopened this Aug 19, 2014
@rubenjgarcia rubenjgarcia changed the title Debug mongoose queries Debug mongoose queries and Add angular-translate Aug 19, 2014
@lirantal
Copy link
Member

lirantal commented Nov 5, 2014

@rubenjgarcia would be interesting to see it as a package indeed. anyway though, I'd remove the mongoose debug which isn't related at all to all the i10n focused PR.

@ilanbiala
Copy link
Member

@rubenjgarcia can you split this PR into 2?

}
},
mongoose: {
debug: true
Copy link
Member

Choose a reason for hiding this comment

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

I think we should make it db: {} and put the db key with the url under this object as url.

@ilanbiala
Copy link
Member

@rubenjgarcia can you pull out the mongoose debugging into a separate PR? We'll merge that into 0.4.0 because it's a good idea.

@ilanbiala ilanbiala self-assigned this Jan 25, 2015
@codydaig
Copy link
Member

codydaig commented Aug 2, 2015

Since @lirantal took the mongoose debug features from here and did it in #707, can this be closed?

@lirantal
Copy link
Member

lirantal commented Aug 2, 2015

yeah, thanks @codydaig

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants