P5.js-web-editor: Add logging with winston

[siddhant1]

I think we should remove console.logs from the code

Links :- https://dev.to/mornir/-dont-leave-console-logs-in-production-14na
https://hackernoon.com/please-stop-using-console-log-its-broken-b5d7d396cf15

I can send in a PR

Have a look @catarak

[catarak]

totally! it would be amazing to add a logging library. maybe this ticket could be that—investigate and add logging library?

[siddhant1]

Yeah , I will look into it and start refactoring

[siddhant1]

As I was looking through the code , I found out that most of the console statements were used for debugging and some for handling errors.
There are 2 ways to solve this.

1. Remove console.logs used for debugging
   2.find a way to disable console.logs on the prod mode and enable in dev mode which agains boils down to adding a logging library

[catarak]

sounds great to me!

[siddhant1]

@catarak I was thinking of a solution to this ticket.
Should I monkey patch window.console.log which checks if we are in dev mode or prod mode and logs only in dev mode?

[catarak]

process.env.NODE_ENV is already passed to the front end (e.g. in App.jsx), so removing console logs when in prod should be pretty simple!

[GaurangTandon]

@siddhant1 Could I please work on this issue while you're working on the other ones? Thank you!

[siddhant1]

Hey Gaurang , this issue was already solved and needs to be closed. You can work on other ones :)

[catarak]

@siddhant1 / @GaurangTandon this issue hasn't been solved. no logging library has been added.

[siddhant1]

The issue was originally to remove logs which was done , for adding logging library , I will send a pr soon

[sagar-joshi]

I am working on this