247 level based logging #375
Conversation
…ole.log's do not make it to production, and error/critical logs can be stored
… 247-level-based-logging
| @@ -0,0 +1,3 @@ | |||
| export class Constants { | |||
| public static readonly InDev = process.env.NODE_ENV === "development"; | |||
There was a problem hiding this comment.
instead of checking if NODE_ENV === 'development', I'd suggest allowing us to manually configure LOG_LEVEL='warn'|'error'|'info'. This allows us to dynamically change the levels on the API side of things simply by changing the env variable. You can default it to 'INFO' in our .env.sample
Additionally, if this is a server only logger, please put import 'server-only' at the top, if you plan to allow this logger to also be used in the client, we'll need a NEXT_PUBLIC_LOG_LEVEL variable in the front end.
|
My bad leaving those console logs there. They have beeen clean up. |
|
I dont't believe we need a logger for race.tsx as we already have a console.table for the |
|
I'm going to consider thie PR as stale. Feel free to remake it in the future @FragAverage |
Issue #247 | Logging
Discord Username: @FragAverage
What type of PR is this? (select all that apply)
Description
Start of a level based logging system to ensure informational/development logs do not make it to production, and in future error and crititcal logs can be stored or parsed to another system.
Related Tickets & Documents
Added/updated tests?