know-cujojs / know Goto Github PK

CC Share-alike? MIT?

I put this at the bottom of each of my tutorials:

For more cujo.js tutorials, go to http://know.cujojs.com/.

To get started with cujo.js asap, clone it or download it at http://github.com/cujojs/quickstart/.

Does this work? Any suggestions? Maybe the website generator should add this?

There are some wrong semicolon in part of the code (at least twice like in the following code) :

return thisMightFail()
        .otherwise(recoverFromFailure); // HERE 
        .then(alwaysCleanupOnSuccess, alwaysCleanupOnFailure);

http://blog.briancavalier.com/async-programming-part-1-its-messy
http://blog.briancavalier.com/async-programming-part-2-promises

And then write part 3?

Second to last line says:

It's called UMD, Universal Module Format, but that's a topic for our next lesson.

It should be:

It's called UMD, Universal Module Definition, but that's a topic for our next lesson.

JavaScript and Cujo.js in particular have a lot of jargon that is difficult for beginners to grok.

I propose we create a glossaries directory with domain specific files that contain terms with precise but approachable definitions. Links to a spec can provide the gory details where appropriate.

Off the top of my head, we could have a glossary for each of:

• promisses
• modules
• dependency injection
• rest
• javascript
• ...

Typo:

var deps = ids.map(function (id) { return require(id); };

should read:

var deps = ids.map(function (id) { return require(id); });

instead. (Closing ')' around function.)

The "Promises" link is broken here

In the documentation at http://know.cujojs.com/tutorials/modules/consuming-modules-module-ids mentions the following:

// module app/billing/billTo/Customer
define(function (require) {

    // questionable: normalizes to "common/payee/Payee"
    var Payee = require("../../common/payee/Payee");
    ...

Shouldn't it be this instead (note the "app/" prefix in the normalization)?:

...
    // questionable: normalizes to app/common/payee/Payee
    var Payee = require("../../common/payee/Payee");

Really great tutorials by the way!

The link to "create promises for others to consume" is broken.

Something simple that helps people know how/where to submit issues and suggest new tutorials. Should we also put links at the bottom of each tutorial?

I think that one ../ is missing below:

// module app/billing/billTo/Customer
define(function (require) {
    [...]
    // BAD: normalizes to "../util/date" -- a URL, not an ID!
    var Date = require("../../../util/Date");

Should probably read:

// BAD: normalizes to "../util/date" -- a URL, not an ID!
var Date = require("../../../../util/Date");

Also congrats on a really awesome tutorial!