<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Mar 26, 2013, at 9:26 AM, Matthias Wessendorf <<a href="mailto:matzew@apache.org">matzew@apache.org</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><br><br><div class="gmail_quote">On Tue, Mar 26, 2013 at 1:54 PM, Kris Borchers <span dir="ltr"><<a href="mailto:kris@redhat.com" target="_blank">kris@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 style="word-wrap:break-word">Thanks for the review! Comments inline.<div><br><div><div class="im"><div>On Mar 26, 2013, at 3:47 AM, Matthias Wessendorf <<a href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>> wrote:</div>
<br><blockquote type="cite"><div><br></div><div><br></div><div>1) On the left, there is a link to </div><div><a href="http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.html" target="_blank">http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.html</a></div>
<div>not sure if this page should be linked, from the left nav...</div><div>(since it's just a few (out of context?) functions/callbacks)</div></blockquote><div><br></div></div>On the left of what page?</div></div></div>
</blockquote><div><br></div><div><br></div><div>oh - on the right :) (the other left :-))</div><div><br></div><div><h2 style="color:rgb(69,105,139);font-size:24px;margin:20px 0px 3px;font-family:'DejaVu Sans Condensed','Liberation Sans','Nimbus Sans L',Tahoma,Geneva,'Helvetica Neue',Helvetica,Arial,'sans serif';line-height:18px;background-color:rgb(255,255,255)">
<a href="http://staging.aerogear.org/docs/specs/aerogear-js/index.html" style="color:rgb(198,0,15)">Home</a></h2><h3 style="color:rgb(69,105,139);font-size:21px;margin:12px 0px 3px;font-family:'DejaVu Sans Condensed','Liberation Sans','Nimbus Sans L',Tahoma,Geneva,'Helvetica Neue',Helvetica,Arial,'sans serif';line-height:18px;background-color:rgb(255,255,255)">
Namespaces</h3><ul style="font-family:'Lucida Grande','Lucida Sans Unicode',arial,sans-serif;font-size:14px;line-height:17px;padding:0px;margin:0px;list-style-type:none;background-color:rgb(255,255,255)"><li style="margin-top:6px">
<a href="http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.html" style="color:rgb(92,89,84)">AeroGear</a></li></ul></div></div></blockquote><div><br></div>I think I prefer to leave that for now. I would have to hack it out of the nav generation. Let's see if it causes confusion before removing it.<br><blockquote type="cite"><div class="gmail_quote"><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div class="im"><br><blockquote type="cite"><div><br></div><div>2) "var auth3 = AeroGear.Auth( [ "someAuth", "anotherAuth" ] );" </div>
<div> => add some with settings?</div></blockquote><div><br></div></div>Already covered in Google doc.<div class="im"><br><blockquote type="cite"><div> </div><div> </div><div>3) Not sure if it is clear _what_ is supported on the 'settings' (for Pipeline, DataManager, Auth)</div>
<div>For instance, here it's better described what this 'settings', there supports:</div>
<div><a href="http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.Auth.adapters.Rest.html" target="_blank">http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.Auth.adapters.Rest.html</a></div></blockquote>
<div><br></div></div>This is already addressed for Auth. Basically, those settings are determined by the adapter you're using so there is no way to specifically define them on Pipeline, DataManager or Auth. Auth says to refer to the adapter you are using for more details (though there is a typo). I will add that same sentence to the others.</div>
</div></blockquote><div><br></div><div>I did assume that some magic is going on - but being explicit in the doc does not hurt :)</div></div></blockquote><div><br></div>Adding that missing sentence to the others is as explicit as this can get since the adapters is the only place these settings can be defined.<br><blockquote type="cite"><div class="gmail_quote"><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div class="im"><br><blockquote type="cite"><div><br></div><div><br></div>
<div>4) match any example?</div><div><a href="http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.DataManager.adapters.Memory.html" target="_blank">http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.DataManager.adapters.Memory.html</a></div>
</blockquote><div><br></div></div>Thanks! Will add.<div class="im"><br><blockquote type="cite">
<div><br></div><div>5) no filter here:</div><div><a href="http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.DataManager.adapters.SessionLocal.html" target="_blank">http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.DataManager.adapters.SessionLocal.html</a></div>
</blockquote><div><br></div></div>This is a tougher one. There is no code in the sessionLocal adapter for filter since that adapter inherits from Memory. The "Mixes In" section is supposed to imply that the methods</div>
</div></blockquote><div><br></div><div>:) I was imagine that - but would appreciate a concrete sencence - or at least a way to link to that; (on java doc there is an inherited functions section, with clickable links)</div></div></blockquote><div><br></div>I am not seeing a similar construct. The only way I have found to get this to work is by adding unnecessary code which I will not do just for docs. Still looking but doubting there is a solution other than the "Mixes In" section on that page which is unfortunate. :/<br><blockquote type="cite"><div class="gmail_quote">
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"> in Memory are included in sessionLocal unless overridden by sessionLocal. I'll see if there is a way to get that method to show up in docs even though there is no code.<br>
<blockquote type="cite"><div class="im">
<div><br></div><div><br></div><div>6) the read in here is pretty long (one long example)</div><div><a href="http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.Pipeline.adapters.Rest.html" target="_blank">http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.Pipeline.adapters.Rest.html</a></div>
<div><br></div></div><div>I'd prefer several smaller examples…</div></blockquote><div><br></div>I can look into breaking that up.<br></div></blockquote><div><br></div><div>+1</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><blockquote type="cite"><div class="im"><div><br></div><div><br></div><div>BTW... I like the links to the 'types' of the callbacks!</div><div><br></div><div> </div><br>
<div class="gmail_quote">
On Mon, Mar 25, 2013 at 10:32 PM, Kris Borchers <span dir="ltr"><<a href="mailto:kris@redhat.com" target="_blank">kris@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 style="word-wrap:break-word">Please take a look at <a href="http://staging.aerogear.org/docs/specs/aerogear-js/" target="_blank">http://staging.aerogear.org/docs/specs/aerogear-js/</a> and let me know if you would like anything else updated, keeping in mind that we are working on <a href="https://docs.google.com/document/d/17J-qFm_Fr0rsgusgjcwGhXFOnd2369KZgOYP9kkwOlQ/edit" target="_blank">https://docs.google.com/document/d/17J-qFm_Fr0rsgusgjcwGhXFOnd2369KZgOYP9kkwOlQ/edit</a><div>
<br></div><div>Thanks!</div></div><br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a 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 href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
</div>
twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
_______________________________________________<div class="im"><br>aerogear-dev mailing list<br><a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a></div>
</blockquote></div><br></div><br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
<a 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 href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
_______________________________________________<br>aerogear-dev mailing list<br><a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>https://lists.jboss.org/mailman/listinfo/aerogear-dev</blockquote></div><br></body></html>