Skip to content

Pass arguments to AJAX_SUCCESS and AJAX_FAIL#118

Open
xfra35 wants to merge 1 commit into
kylefox:masterfrom
xfra35:ajax_args
Open

Pass arguments to AJAX_SUCCESS and AJAX_FAIL#118
xfra35 wants to merge 1 commit into
kylefox:masterfrom
xfra35:ajax_args

Conversation

@xfra35

@xfra35 xfra35 commented Aug 28, 2015

Copy link
Copy Markdown
Contributor

Usage:

$('#ajax-link').on('modal:ajax:fail',function(event,jqXHR,textStatus,errorThrown){
  //analyze here why ajax failed
});

@xfra35

xfra35 commented Dec 2, 2016

Copy link
Copy Markdown
Contributor Author

🆙

Here's a use case: to give the ability to customize error messages according to the HTTP status (404, 403, 401, 400, 500).

@kylefox

kylefox commented Dec 12, 2016

Copy link
Copy Markdown
Owner

You're right, this is a good idea — if you're able to get it to merge cleanly I'll be happy to pull it in 👍

@kylefox

kylefox commented Dec 12, 2016

Copy link
Copy Markdown
Owner

BTW sorry I missed your original PR, not sure how/why I didn't see it!

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.

2 participants