-
Notifications
You must be signed in to change notification settings - Fork 33
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
feat: my attempt to support more loggers, other than just console.log #62
feat: my attempt to support more loggers, other than just console.log #62
Conversation
…. Implements feature request #53
i.e. the user has to specify the `logger` function. Defaults to: `console.log`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this solution should keep everyone happy, hope you guys like it and that we can avail of v2.4.0 as soon as possible :-) Cheers!
@@ -116,13 +116,17 @@ instance.interceptors.request.use((request) => { | |||
|
|||
#### Enable config list | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see https://github.com/ClintEsteMadera/axios-logger/tree/add-support-for-other-loggers-than-just-console-log#enable-config-list to appreciate how this will look like. I think it's a tad cleaner this way...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good table!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@ClintEsteMadera I will merge tomorrow. |
@hg-pyun awesome! Please do let me know when the new release is published in NPM...thanks! |
Hi, I'm sending this contribution for your consideration. I would be really happy if this feature could be published in the library as soon as possible as I prefer not having to publish my own version in my private NPM repo...
This is the first time I contribute to an open source project so any feedback will be appreciated!
Attempts to implement the feature request #53