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 rid of the spam. Can be found in the network tab. #133

Merged
merged 1 commit into from
Jun 25, 2015

Conversation

TimvdLippe
Copy link
Contributor

No description provided.

@eanker
Copy link
Contributor

eanker commented Jun 25, 2015

LGTM, trivial

@jwgmeligmeyling
Copy link
Member

Can't do this, the absence of this familiar log statement would break my heart 💔

TimvdLippe pushed a commit that referenced this pull request Jun 25, 2015
Get rid of the spam. Can be found in the network tab.
@TimvdLippe TimvdLippe merged commit e6c1ed2 into master Jun 25, 2015
@TimvdLippe TimvdLippe deleted the fetchMessagesLogRemoval branch June 25, 2015 11:56
@jwgmeligmeyling
Copy link
Member

How cold you

@jwgmeligmeyling
Copy link
Member

TBH It would be better to keep the log statements, but turn the debug related logs off in production:

module.config(function($logProvider){
  $logProvider.debugEnabled(false);
});

@TimvdLippe
Copy link
Contributor Author

Yes, let's make a seperate issue for that.

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

Successfully merging this pull request may close these issues.

None yet

3 participants