Re: [aerogear-dev] [iOS] Auth: API for login, signup and logout

Hi, For what is worth, I prefer the current approach with the two callbacks. Kind of more familiar with that pattern. Regards, Christos On 6 Οκτ 2012, at 1:51 μ.μ., Matthias Wessendorf <matzew(a)apache.org&gt; wrote: > Hey, > > I tend to continue using two callbacks: One for 'success' and one for > the error case. > > Any objections? > > Greetings, > Matthias > > On Fri, Oct 5, 2012 at 9:26 AM, Matthias Wessendorf <matzew(a)apache.org&gt; wrote: >> On Fri, Oct 5, 2012 at 9:09 AM, Elmar Kretzer <elkr(a)symentis.com&gt; wrote: >>> Hi Matze, =) >>> >>> quite fast enroll of the first API version - congratulations. >> >> it's the third :) >> our Pipe API is already here: >> https://github.com/aerogear/aerogear-ios/blob/master/API.md >> >> The datamanager proposal is here: >> http://aerogear-dev.1069024.n5.nabble.com/aerogear-dev-iOS-DataManager-St... >> >>> >>> I just had a quick look at your gists - and i then i stumbled upon the >>> method with 2 blocks. >>> >>> It`s more like a personal flavor thing but sometimes methods with 2 block >>> params can look quite disturbing. So i tend to use up to 3 params if >>> necessary. >> >> I was following the pattern from the AFNetworking folks, since I used >> that underneath our 'http communication' facility; >> (see pipeline/pipe API) >> >> >> I think I generally don't mind one single block - but instead of >> completion, I'd prefer completionHandler :) >> Not sure... >> >> >>> I forked your gist for better explanation: https://gist.github.com/3838469 >>> >>> As i said - it`s really more a personal flavor thing - don`t mind. >>> So keep up the work - going to have a deeper look and if i got time i will >>> make a test. >> >> >> Elmar, thanks for the feedback >> >> >> -Matthias >> >> >>> >>> Cheers Elmar >>> >>> >>> >>> >>> >>> -- >>> View this message in context: http://aerogear-dev.1069024.n5.nabble.com/aerogear-dev-iOS-Auth-API-for-l... >>> Sent from the aerogear-dev mailing list archive at Nabble.com. >>> _______________________________________________ >>> aerogear-dev mailing list >>> aerogear-dev(a)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 > > > > -- > Matthias Wessendorf > > blog: http://matthiaswessendorf.wordpress.com/ > sessions: http://www.slideshare.net/mwessendorf > twitter: http://twitter.com/mwessendorf > _______________________________________________ > aerogear-dev mailing list > aerogear-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/aerogear-dev _______________________________________________ aerogear-dev mailing list aerogear-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev