-
Notifications
You must be signed in to change notification settings - Fork 96
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
Create a Consola instance in utils/logger.ts
#61
Comments
I'd like to work on this issue. @amandesai01 |
go ahead, give it a try. |
@VenkyGitRep any updates? you still working? feel free to share your implementation via draft pr. |
I've managed to get things running in my local yesterday. I had an interview last night, and had to pause work on this issue for a bit. I'll be working on setting up console today, will share an update by the evening. |
@amandesai01 Can we have a quick chat somewhere? |
you can join discord from readme. |
@VenkyGitRep you want me to take over? or wanna give it a try? |
Okay, go head. The changes I've made are in |
We don't want logger in frontend. You can refer parent issue. |
sure, I can chew on this later today. I think I can have this working. Thanks for the direction, I'll get back when I have some questions. |
I've now submitted a PR. |
@amandesai01 What do you think? I've submitted a new PR. |
@VenkyGitRep changes are good so far, but we still need some research. We want logger to have a dynamic tag with request path. And when we log something, it should also log request path. This will help in debugging by helping us identify what exactly happened with the request. Problem statement is weird and I don't know if that is possible in Nitro yet. But a research on this is necessary. |
Also for each request we would need to generate a unique id so while seeing the logs we will be able to understand the flow |
Let's research on this, as this is the next target. |
I cleared out assignees until we have strong implementation proposal. |
No description provided.
The text was updated successfully, but these errors were encountered: