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

docs($http): Adds detail about param sorting. #11745

Closed
wants to merge 1 commit into from

Conversation

wesalvaro
Copy link
Contributor

Parameters are sorted when added to the URL:

forEachSorted(params, function(value, key) {

This is a very important detail, especially for testing.

@Narretz Narretz added this to the Backlog milestone Apr 28, 2015
@Narretz
Copy link
Contributor

Narretz commented Apr 28, 2015

Actually, this behavior isn't even tested. We should probably test this, too.

@pkozlowski-opensource
Copy link
Member

And with the custom param serializers we should probably move this doc to a serializer itself...

@pkozlowski-opensource pkozlowski-opensource self-assigned this Apr 28, 2015
Narretz added a commit to Narretz/angular.js that referenced this pull request Jun 9, 2015
@Narretz Narretz closed this in 0400dc9 Jun 14, 2015
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
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.

4 participants