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

fix($http): add support for PATCH short method#6071

Closed
dagingaa wants to merge 1 commit into
angular:masterfrom
dagingaa:fix_http_add_patch_short_method_back
Closed

fix($http): add support for PATCH short method#6071
dagingaa wants to merge 1 commit into
angular:masterfrom
dagingaa:fix_http_add_patch_short_method_back

Conversation

@dagingaa
Copy link
Copy Markdown

Commit 46691c2 removed the short method for PATCH from $http because there were too many unknowns. However, #5823 schedules PATCH to be re-added for the 1.3.x milestone.

This change adds the short method for PATCH back to the $http library, including tests.

Closes #5823

Commit 46691c2 removed the short method
for PATCH from $http because there were too many unknows. However, angular#5823
schedules PATCH to be re-added for the 1.3.x milestone.

This change adds the short method for PATCH back to the $http library,
including tests.

Closes angular#5823
@dagingaa
Copy link
Copy Markdown
Author

God damnit, my brain farted, and I didn't see the other PR #5894. That one includes data in the tests, which is correct according to the spec (I just manually reverted 46691c2 blindly). Closing this.

@dagingaa dagingaa closed this Jan 31, 2014
@dagingaa dagingaa deleted the fix_http_add_patch_short_method_back branch January 31, 2014 12:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add the PATCH shortcut back to $http

1 participant