thenativeweb / cqrs-sample

CQRS, EventSourcing, (DDDD) Sample in node.js
148 stars 50 forks source link

Trying to upgrade cqrs sample #13

Closed pawarvijay closed 8 years ago

pawarvijay commented 8 years ago

Need Assistance in upgrading cqrs_sample

link to crqs-sample upgrade that i tried

Change List : 1 : updated cqrs-domain , cqrs-eventdenormalizer , viewmodel to latest 2 : corresponding to above 3 core modules i also updated configuration code in server.js by referencing to nice documentation in each modules.

General doubt 1 : Was It the right way to upgrade your repository( means upgrading module and configuration code in server.js ) 2 : Is it ok to use these modules in production . . or there are major updates yet to come .

Code doubt 1 : In domain/server.js file

code snippet from node-cqrs-domain documentation

`domain.onEvent(function (evt, callback) {

     msgbus.emit('event', evt, function ack () {

        callback();
    });

});`

In msgbus.js there is not emit function ..?

2 : In host/server.js file

code snippet from node-cqrs-eventdenormalizer documentation

 `eventDenormalizer.onEvent(function (evt, callback) {

    msgbus.emit('event', evt, function ack () {

            callback();

        });

    });`

In msgbus.js there is not emit function ..?

link to crqs-sample upgrade that i tried

adrai commented 8 years ago

can you try again... i fixed and updated the dependencies...

pawarvijay commented 8 years ago

now its working nicely