[aerogear-dev] [JS Doc] feedback on the new JS-Doc
Matthias Wessendorf
matzew at apache.org
Tue Oct 9 03:29:06 EDT 2012
Hi Kris,
overall I like the new JS doc (see [1]). Looks pretty nice! However,
here is some feedback on the new JS API doc!
* http://staging.aerogear.org/docs/specs/aerogear-js/AeroGear.html
- add/remove/ajax: These are internal helpers/utilities, right? Not
sure if that should be exposed...
I think the 'isArray' is a bit different => I can see that some
END-USER finds 'AeroGear.isArray(argument)' pretty useful!
(compared to something like AeroGear.ajax(...), where you already need
to pass in an AeroGear object)
* On the different adapters:
Regarding their privileged functions => Do we want to expose/document
the privileged methods? Or 'just' the public methods?
Fixed a little typo:
https://github.com/aerogear/aerogear-js/commit/3ed416af824d2b398a7db8b276ca46bf66fc2603#src/pipeline/aerogear.pipeline.js
Added a PR for two very simple fixes on the JS lib:
https://github.com/aerogear/aerogear-js/pull/6
Greetings,
Matthias
[1] http://staging.aerogear.org/docs/specs/aerogear-js/
--
Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf
More information about the aerogear-dev
mailing list