-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
[Feature request] add loadstart event #1859
Comments
Agreed. I think often you really don't need this, since you will trigger those kind of things (spinners etc.) on the event which triggers the request. But I can see use cases where you really want to trigger it on the As you already mentioned a callback on the data object would do the job. Something like this:
Are you going to implement this? Otherwise I would probably find a some time by the end of this week to do so. |
Hey @Orlandster, I'm not the best with js so it's probably better if you start the PR. Cheers |
@jimblue Sounds good. I'll keep you updated. |
Waiting for this... |
2 similar comments
Waiting for this... |
Waiting for this... |
Waiting for this... |
See #4209 |
It could be nice to add XMLHttpRequest loadstart event inside Axios.
There is actually no event callback when the request start that could be use to display a loading spinner.
Thanks
The text was updated successfully, but these errors were encountered: