Angular update method not working

Question

Angular update method not working

bjartek opened this issue 11 years ago · comments

Bjarte S. Karlsen commented 11 years ago

Angular's ngResources uses a POST request to run the update operation but that is not supported by the routes in AutoSourceController.

PUT is the correct method to use here IIRC, but it would be nice if POST with an id was supported as well since it would make it easer to use the framework with angular.

Something like:
case ("POST", Id(id)) => withId(id, update)

in AutoSourceController

Bjarte S. Karlsen · Answer 1 · Mon Sep 16 2013 20:54:14 GMT+0800 (China Standard Time)

Another way to get this to work is using the my.resource from this article: http://kirkbushell.me/angular-js-using-ng-resource-in-a-more-restful-manner/

That is IMHO a better way since the API is then correct and it works seamlessly.

Pascal Voitot · Answer 2 · Mon Sep 16 2013 21:08:22 GMT+0800 (China Standard Time)

completely agree!
I prefer having a purer API.
Thanks for contrib!
Do you use Autosource ?

On Mon, Sep 16, 2013 at 2:54 PM, Bjarte S. Karlsen <notifications@github.com

wrote:

Another way to get this to work is using the my.resource from this
article:
http://kirkbushell.me/angular-js-using-ng-resource-in-a-more-restful-manner/

That is IMHO a better way since the API is then correct and it works
seamlessly.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/9#issuecomment-24507423
.

Bjarte S. Karlsen · Answer 3 · Mon Sep 16 2013 21:09:31 GMT+0800 (China Standard Time)

I am just playing around with autosource atm. Will probably use it in some free time projects for now.

Pascal Voitot · Answer 4 · Mon Sep 16 2013 21:13:35 GMT+0800 (China Standard Time)

cool!
I expect to release a 1.0 version when Play 2.2 is out... then each DB
implementation will evolve on 1.x custom versions

On Mon, Sep 16, 2013 at 3:09 PM, Bjarte S. Karlsen <notifications@github.com

wrote:

I am just playing around with autosource atm. Will probably use it in some
free time projects for now.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/9#issuecomment-24508259
.

Bjarte S. Karlsen · Answer 5 · Mon Sep 16 2013 21:14:36 GMT+0800 (China Standard Time)

Nice! Keep up the good work.

Bjarte S. Karlsen · Answer 6 · Mon Sep 16 2013 21:36:39 GMT+0800 (China Standard Time)

Come to think of it, should we not modify the example in the README so that it mentions this? I can try to send a Pull Request later today if you do not have time to fix it.