[aerogear-dev] Android Guides

[Summers Pittman]

On 03/20/2013 04:44 AM, Matthias Wessendorf wrote:
> Howdy,
>
> great to see some docs/guides
>
> Here are my first/initial comments:
>
> 1) Group them into a subfolder, like daniel did, for the controller
+1
>
> 2) links to the matching JavaDocs (classes (e.g. PipeHandlert etc)) 
> does not hurt
+1
>
> 3) not sure about if the `AerogearAndroidCustomizingPipes.asciidoc` is 
> really useful ...
I disagree.  It may be titled wrong, but knowing how the Android library 
implements Pipes and how to extend this implementations for your own end 
is quite useful.

At least that is the goal of the doc, what did you see differently?

>
> "AerogearAndroidPagination"
> 4) You never show ParameterProvider...
Because the default implementation of Parameter Provider just consumes 
data provided by the PageConsumer.  Explicitly setting it isn't necessary.

> 5) what is config.setDataRoot()
The root of the data you want to operate on as explained in the Intro 
document and the API's.

For example, in the DevNexus result set for speakers you have

{
     speakers :  {
         data:[
             {speaker1},
             {speaker2}
         ]
     }
}

The "list" of data you are asking for actually starts at the data 
parameter of the speakers parameter. 
config.setDataRoot("speakers.data"); makes this explicit.


> --> looks like the Android bits are a bit _different_, compared to the 
> iOS/JS version
> 6) do I really have to implement my own "PageResultExtractor"
No unless you are consuming it from a service where the paging doesn't 
follow "simple" patterns.
> The Pagination feels: Lot's of code... for a simple read...
>
Because the simple read of pagination is linked to at the beginning for 
the doc.  This is for consuming arbitrary paging from a site. Perhaps 
this needs to be an "advanced" doc and I should discuss simpler paging 
in an into doc.  Like Pipes.
>
>
> -Matthias
Seems like some of the docs might be missing things (an intro read to 
paging, more explicit goals of the guid etc).
>
>
> On Tue, Mar 19, 2013 at 9:19 PM, Summers Pittman <supittma at redhat.com 
> <mailto:supittma at redhat.com>> wrote:
>
>     I just committed some guides for using the Aerogear Android
>     libraries.
>
>     https://github.com/aerogear/aerogear.org/commit/d3cae265d
>
>
>     I'm still editing them for gramma/spelling/formatting, so be
>     warned (hence why I havn't PRed it yet).  Also, if there is
>     anything I missed, glossed over, explained like crap let me know
>     and I will try to reword it when I begin proofreading.
>
>     Summers
>
>     _______________________________________________
>     aerogear-dev mailing list
>     aerogear-dev at lists.jboss.org <mailto: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/20130321/97b99e91/attachment.html