Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

fix($httpBackend): don't error when JSONP callback called with no parameter #7031

Closed
wants to merge 1 commit into from

Conversation

caitp
Copy link
Contributor

@caitp caitp commented Apr 7, 2014

This change brings Angular's JSONP behaviour closer in line with jQuery's. The feature has
already landed in the 1.3 branch as 6680b7b, however this alternative version is intended
to implement the feature in an IE8-compatible fashion.

@caitp caitp added this to the 1.3.0-beta.6 milestone Apr 7, 2014
@caitp caitp self-assigned this Apr 7, 2014
@caitp
Copy link
Contributor Author

caitp commented Apr 7, 2014

This is a bit broken, and I don't think I'll get it fixed today, but I'll try to have this done before the end of the week, and minimize the differences with the 1.3 branch

@caitp caitp added cla: yes and removed cla: no labels Apr 9, 2014
…ameter

This change brings Angular's JSONP behaviour closer in line with jQuery's. The feature has
already landed in the 1.3 branch as 6680b7b, however this alternative version is intended
to implement the feature in an IE8-compatible fashion.
@IgorMinar
Copy link
Contributor

lgtm

caitp added a commit that referenced this pull request Apr 15, 2014
…ameter

This change brings Angular's JSONP behaviour closer in line with jQuery's. The feature has
already landed in the 1.3 branch as 6680b7b, however this alternative version is intended
to implement the feature in an IE8-compatible fashion.

Closes #7031
@caitp
Copy link
Contributor Author

caitp commented Apr 15, 2014

Closed by a7ccb75

@caitp caitp closed this Apr 15, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants