Skip to content
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

Removing the IE8 PATCH requests fix #3172

Merged
merged 1 commit into from
Jun 2, 2014
Merged

Conversation

gmajoulet
Copy link
Contributor

Patch requests were not working on IE8, even with the Backbone fix (tested on XP/IE8 with modern.ie VMs).

Remove this line when jQuery supports `PATCH` on IE8.

jQuery 1.11.* and the corresponding 2.x version now support PATCH requests (see: http://bugs.jquery.com/ticket/13240)

It now works as expected with jQuery

@braddunbar
Copy link
Collaborator

👍 Fine by me, though perhaps we should also document the version of jQuery we depend upon.

akre54 added a commit that referenced this pull request Jun 2, 2014
Removing the IE8 PATCH requests fix
@akre54 akre54 merged commit a32e1ff into jashkenas:master Jun 2, 2014
@akre54
Copy link
Collaborator

akre54 commented Jun 2, 2014

Looks good by me. Glad jQuery finally came around.

78fad22

@stavarotti
Copy link

I agree with @braddunbar unless it is assumed that folks should be using the latest version of jQuery. Either way perhaps updating the main docs too along with this pr?

@jashkenas
Copy link
Owner

perhaps we should also document the version of jQuery we depend upon

Yes. Very important.

@braddunbar
Copy link
Collaborator

Yep yep, @akre54 already updated in 78fad22. 😃 Thanks Adam!

@akre54
Copy link
Collaborator

akre54 commented Jun 2, 2014

It'll go out with the next release when this patch lands. Do we need
something more explicit in the change log?

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.

5 participants