Skip to content

Wrap $.ajax promise api ... #337

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

Closed
wants to merge 1 commit into from
Closed

Conversation

benvinegar
Copy link
Contributor

fixes #326

@benvinegar
Copy link
Contributor Author

Not sure how best to test this, or if there are any plugin tests to begin with.

@mattrobenolt
Copy link
Contributor

How does this relate to #268 ?

@benvinegar
Copy link
Contributor Author

#268 is a broader, better solution, IMHO. It also covers cases where jQuery library users choose to use $.Deferred themselves.

@benvinegar benvinegar closed this Mar 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Errors in jQuery AJAX callbacks using promise syntax are not caught
2 participants