Modella 0.2 support and ajaxParseAllBody and ajaxParseGetBody events

[alexmingoia]

Modella 0.2 support and ajaxParseAllBody and ajaxParseGetBody events. Closes modella/ajax/7.

[rschmukler]

@alexmingoia you rock dude! I'm going to try and do mongo tonight or tomorrow and ship 0.2.0 later this week.

[scttnlsn]

Nice! What about some event hooks for saving/updating/deleting? It's possible that reusing the ajaxParseGetBody makes sense here.

@rschmukler You mentioned a plugin writing guide here: modella/modella#21 (comment) Is that still in the works?

[alexmingoia]

@scttnlsn Hooks for saving and deleting are not specific to the ajax sync module, they are provided by modella core.

See:
https://github.com/modella/modella/blob/0.2.0-pre/lib/proto.js#L118-L122
https://github.com/modella/modella/blob/0.2.0-pre/lib/proto.js#L158-L163
https://github.com/modella/modella/blob/0.2.0-pre/lib/proto.js#L158-L163

[rschmukler]

@scttnlsn Yeah, it is. Sorry, I've been working on something else these last few days (which I'll be releasing tonight, if all goes well!) but then I will.

@alexmingoia I think @scttnlsn is saying that having something like having ajaxParseSingleBody instead of ajaxParseGetBody makes sense. And then we use that on response from PUT /users/:id and POST /users/ etc for an opportunity to parse response.

[scttnlsn]

@rschmukler Awesome! And yes, that's what I was imagining...something to parse the responses from PUT and POSTs to the resource.

[alexmingoia]

Agreed we need better event coverage. I propose emitting the event ajax :method_name response for every response. I also propose the space delimited name for easier parsing and conformity with other events.

Example

User.on('ajax get response', function(res) {
    res.body.registeredAt = new Date(res.body.registeredAt);
});

User.on('ajax all response', function(res) {
    var users = res.body.results;
    // Convert JSON string dates into actual dates
    users.forEach(u) {
       u.registeredAt = new Date(u.registeredAt);
    }
    res.body = users;
});


User.on('ajax removeAll response', function(res) {
    // ..
});

User.on('ajax save response', function(res) {
    // ..
});

User.on('ajax update response', function(res) {
    // ..
});

User.on('ajax remove response', function(res) {
    // ..
});

I have time to do this tomorrow afternoon. Thoughts?

[rschmukler]

@alexmingoia What do you think about dropping the response part? So it becomes ajax update instead of ajax update response... Makes it a bit more concise but maybe a little less descriptive. What do you think?

[alexmingoia]

@rschmukler Sounds good. I'm fine with that.

[rschmukler]

Awesome. If you have time to do it that's awesome. I'm going to try for modella-mongo tomorrow. Today was dedicated to agenda.

[scttnlsn]

+1 on the shorter event names ajax get, ajax all, ajax save, etc.