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

Improve logging to contain ids #80

Open
leichter opened this issue Jun 7, 2016 · 0 comments
Open

Improve logging to contain ids #80

leichter opened this issue Jun 7, 2016 · 0 comments
Assignees

Comments

@leichter
Copy link
Contributor

leichter commented Jun 7, 2016

May be something of a joint effort:

it seems like often, when things go either right or wrong in watchman, watchman doesn't log out the ids of the entities its dealing with. This can make it difficult to deal with issues; for example, I can't really track down why watchman might "Fail to increase campaign budget" if I don't have the id of the campaign or anything.

I feel like when in doubt, it's better to err on the side of caution when it comes to logging and log out as many relevant ids as you can.

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