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.

Sign up for GitHub

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

Jump to bottom

Explore changing logging #40

Closed
neilenns opened this issue Jul 4, 2020 · 0 comments · Fixed by #48
Closed

Explore changing logging #40

neilenns opened this issue Jul 4, 2020 · 0 comments · Fixed by #48

Comments

@neilenns
Copy link
Owner

neilenns commented Jul 4, 2020

Right now console.log is used everywhere instead of this.error which appears to be the more correct way to do it. Explore changing methods.
neilenns added a commit that referenced this issue Jul 4, 2020
@neilenns
Update some of the logging
cd91cc0 
Fixes #40
@neilenns neilenns mentioned this issue Jul 4, 2020
Closed
neilenns added a commit that referenced this issue Jul 5, 2020
@neilenns
Explore changing logging
3ab5ca7 
Fixes #40
@neilenns neilenns mentioned this issue Jul 5, 2020
Merged
1 task
neilenns added a commit that referenced this issue Jul 5, 2020
@neilenns
Update logging to use NodeRed methods (#48)
862ff66 
* Update some of the logging
Fixes #40

* Update logging to use NodeRed methods

* Enforce no console.log via eslint

* Add missing exception handlers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update logging to use NodeRed methods neilenns/node-red-contrib-wled2
1 participant
@neilenns