Merge lp://staging/~abreu-alexandre/unity-webapps-qml/fix-oa-docs into lp://staging/unity-webapps-qml

please fix tabs to be spaces for indenting

please add a description of the OnlineAccounts object. maybe something like: An object that represents all online accounts currently on the system.

alex-abreu, AccountsService has a "@constructor' that takes to params but there are no @param tags.

Also, @constructor means the user creates a new object with 'new', like var as = new AccountService(...) Is that what you expect? or does an AccountService get returned to the user (with no 'new' usage)

for every method that returns something, lets always start the description like this:
"Returns ...."

Not 'retrieves'. Not implied.

For example, this should be "Returns..."
        /**
         * Retrieves the account's numeric ID; note that all
>. * AccountService objects which work on the same online account will have the same ID.
         *
         * If the callback parameter is not set, the current "local" value is retrieved.
         *
         * @method accountId
         * @param callback (optional) {Function(String)}
         */
        accountId: function(callback) {

PASSED: Continuous integration, rev:93
http://jenkins.qa.ubuntu.com/job/unity-webapps-qml-ci/78/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-webapps-qml-trusty-amd64-ci/26
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-webapps-qml-trusty-armhf-ci/26

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-webapps-qml-ci/78/rebuild

AND to follow up on that last comment, you need to consistently add the @return tag:
http://yui.github.io/yuidoc/syntax/#return

please let me know when these are done and I will scan again. thx

thanks for the work, for example indents.

Please fix the tab indents you just added though. :)

Now we have "@return null"s

if a method does not return anything, maybe don't have the @return tag at all (unless it is important for some reason to show that a literal null IS returned...)

so as discussed, let's remove the optional callback args for now to be replaced later with signals for mutable QML values.

good to go

approve