Skip to content
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Patch for Rest - #1587

Merged
WonderCsabo merged 6 commits into
androidannotations:developfrom
simopete:1333_Patch_for_Rest
Oct 10, 2015
Merged

WonderCsabo merged 6 commits into
androidannotations:developfrom
simopete:1333_Patch_for_Rest

Conversation

@simopete

Copy link
Copy Markdown
Contributor

Implements #1333

Some test are missing because PATCH is only available in Spring Android Rest Template 2.0

WonderCsabo added a commit that referenced this pull request Oct 10, 2015
@WonderCsabo
WonderCsabo merged commit 873f6d1 into androidannotations:develop Oct 10, 2015
@WonderCsabo WonderCsabo added this to the 4.0 milestone Oct 10, 2015
@WonderCsabo

Copy link
Copy Markdown
Member

I decided to merge this PR without explicit tests for @Patch. Actually that is just like POST, so its functionality is already tested, and this PR also paves the way for other features (@QueryMap, requreid annotations for all method params). We could switch our test suite to use Spring Android 2.0, however i am not sure about that, because it is still not released, and we cannot use milestone or snapshot dependencies. @yDelouis WDYT?

@WonderCsabo

Copy link
Copy Markdown
Member

@simopete great work, thanks! Can you update the wiki? Please note the Part and Field annotations should be moved to a new section.

@WonderCsabo

Copy link
Copy Markdown
Member

Wiki merged, thanks!

@simopete
simopete deleted the 1333_Patch_for_Rest branch March 20, 2016 14:12
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.

3 participants