[aerogear-dev] Push Endpoints

[Lucas Holmquist]

On Jun 27, 2013, at 8:03 AM, Matthias Wessendorf <matzew at apache.org> wrote:

> 
> 
> 
> On Thu, Jun 27, 2013 at 2:00 PM, Sebastien Blanc <scm.blanc at gmail.com> wrote:
> Hum for both for me it should be an 200 OK with no reponse body. For the update is we have a 200 OK , it is the responsability of the client to do a GET to retrive the updated client 
> Agree on the GET 
Thats fine,  i wasn't sure what was the "correct" way of doing it.  I'll create a JIRA to update the response to return OK.  should be a pretty easy PR


> 
> 
> On Thu, Jun 27, 2013 at 1:57 PM, Lucas Holmquist <lholmqui at redhat.com> wrote:
> I've been looking at the endpoints of the push server a little more since coding the Admin UI, and i see that for "updates" and "deletes", the response is "no content"
> 
> for example:
> https://github.com/aerogear/aerogear-unified-push-server/blob/master/src/main/java/org/jboss/aerogear/connectivity/rest/registry/applications/PushApplicationEndpoint.java#L117
> 
> 
> I wasn't sure what we should be returning here.
> 
> I'd imagine for update we would want to return the update entity
> 
> but what about for delete?  In the case of deleting a "Push Application",  should we return the new list of Push Applications?
> 
> 
> thoughts?
> 
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev
> 
> 
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev
> 
> 
> 
> -- 
> Matthias Wessendorf 
> 
> blog: http://matthiaswessendorf.wordpress.com/
> sessions: http://www.slideshare.net/mwessendorf
> twitter: http://twitter.com/mwessendorf
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20130627/cf9bc04f/attachment.html