<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 03/20/2013 04:44 AM, Matthias
Wessendorf wrote:<br>
</div>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">Howdy,
<div><br>
</div>
<div>great to see some docs/guides</div>
<div><br>
</div>
<div>Here are my first/initial comments:</div>
<div><br>
</div>
<div>
<div>1) Group them into a subfolder, like daniel did, for the
controller</div>
</div>
</blockquote>
+1<br>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">
<div>
<div>
<br>
</div>
<div>2) links to the matching JavaDocs (classes (e.g.
PipeHandlert etc)) does not hurt</div>
</div>
</blockquote>
+1<br>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">
<div>
<div><br>
</div>
<div>3) not sure about if the
`AerogearAndroidCustomizingPipes.asciidoc` is really useful
...</div>
</div>
</blockquote>
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.<br>
<br>
At least that is the goal of the doc, what did you see differently?<br>
<br>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">
<div>
<div>
<br>
</div>
<div>"AerogearAndroidPagination"</div>
<div>4) You never show ParameterProvider...</div>
</div>
</blockquote>
Because the default implementation of Parameter Provider just
consumes data provided by the PageConsumer. Explicitly setting it
isn't necessary.<br>
<br>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">
<div>
<div>5) what is config.setDataRoot()</div>
</div>
</blockquote>
The root of the data you want to operate on as explained in the
Intro document and the API's.<br>
<br>
For example, in the DevNexus result set for speakers you have <br>
<br>
{<br>
speakers : {<br>
data:[<br>
{speaker1}, <br>
{speaker2}<br>
]<br>
}<br>
}<br>
<br>
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.<br>
<br>
<br>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">
<div>
<div>--> looks like the Android bits are a bit _different_,
compared to the iOS/JS version </div>
<div>6) do I really have to implement my own
"PageResultExtractor" </div>
</div>
</blockquote>
No unless you are consuming it from a service where the paging
doesn't follow "simple" patterns.<br>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">
<div>
<div>The Pagination feels: Lot's of code... for a simple read...</div>
<div><br>
</div>
</div>
</blockquote>
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.<br>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">
<div><br>
</div>
<div><br>
</div>
<div>-Matthias</div>
</blockquote>
Seems like some of the docs might be missing things (an intro read
to paging, more explicit goals of the guid etc).<br>
<blockquote
cite="mid:CAAg5f2TQ1KC5WdZ9SnLxn9m_Ra08ghexPuXTkBT589DAvEJoMQ@mail.gmail.com"
type="cite">
<div><br>
<div><br>
<div class="gmail_quote">On Tue, Mar 19, 2013 at 9:19 PM,
Summers Pittman <span dir="ltr"><<a
moz-do-not-send="true" href="mailto:supittma@redhat.com"
target="_blank">supittma@redhat.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000"> I just committed
some guides for using the Aerogear Android libraries. <br>
<br>
<a moz-do-not-send="true"
href="https://github.com/aerogear/aerogear.org/commit/d3cae265d"
target="_blank">https://github.com/aerogear/aerogear.org/commit/d3cae265d</a><br>
<br>
<br>
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.<span class="HOEnZb"><font
color="#888888"><br>
<br>
Summers<br>
</font></span></div>
<br>
_______________________________________________<br>
aerogear-dev mailing list<br>
<a moz-do-not-send="true"
href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
<a moz-do-not-send="true"
href="https://lists.jboss.org/mailman/listinfo/aerogear-dev"
target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
</blockquote>
</div>
<br>
<br clear="all">
<div><br>
</div>
-- <br>
Matthias Wessendorf <br>
<br>
blog: <a moz-do-not-send="true"
href="http://matthiaswessendorf.wordpress.com/"
target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
sessions: <a moz-do-not-send="true"
href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
twitter: <a moz-do-not-send="true"
href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
</div>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
aerogear-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a>
<a class="moz-txt-link-freetext" href="https://lists.jboss.org/mailman/listinfo/aerogear-dev">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a></pre>
</blockquote>
<br>
</body>
</html>