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

Add XHR failed object when calling this.fireCallbacks('fail',[...])); to... #32

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@slorber
Contributor

slorber commented Feb 5, 2014

... help with issue #29 of rdflib

I didn't add the http status to the args because the most open way to handle this is to provide directly the xhr object.
There may not even be any http status codes for certain errors (like same origin policy errors, timeout errors etc...)

The XHR object contains the URI, the http status code and more informations that can be useful to the callback user. This is not the most "user friendly" but at least it handles all the cases

@timbl

This comment has been minimized.

Member

timbl commented Dec 1, 2014

Agreed. Including future cases.

@dmitrizagidulin dmitrizagidulin force-pushed the linkeddata:master branch from c04068e to 56913a5 Aug 9, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment