js-data / js-data-examples Goto Github PK

Thank you for the hard work :) I think it would be awesome if the rethinkdb server and angular client examples worked with one another—ultimately is that your goal w/ this repo?

Is it possible to commit an example of the container module and associated config referenced in rethinkdb/app.js? container.resolve() seems to be responsible for creating the closure that bootstraps app.js...

var container = require('./container');

Hi,

First of all, thanks for the sample! It's helping me to kickstart a project. However, if someone would run more than one replica of the socket.io server, horrible things would start happening, as instead of using a RethinkDB changefeed - which is one of the main reasons why people use RethinkDB in the first place - it just emits a message with changes which the specific server received. For example,

• user 1 is connected with socket.io to server 1
• user 2 posts data which get load balanced to server 2
• server 2 saves data and emits a socket.io message
• user 1 receives nothing

So all calls like messageService.sendCreateMessage('post', post); should be removed, and socket.io connection needs to be replaced with something like

function on_user_connected( ... ) {
    r.table("posts").changes().run(rConnection).then(function(cursor) {
        cursor.each(function (err, item) {
            ioServer.emit('post created or deleted or updated', JSON.stringify(item, null, 2));
        }
    };
}

I haven't worked up a specific solution yet, but I might post a more specific sample later.

This is the controller with the routing:
https://github.com/js-data/js-data-examples/blob/master/server/sql/app/app.js#L103-L169

And what you do have inside your controllers is actually the Repository.

Also, {any}Service should be called {any}Model, excepting the messageService.

Can't run the server examples because of OAuth errs. Some pointers on setting up the secrets and config would remove friction from checking this stuff out-

/Users/jeffwinkler/repos/js-data-examples/server/rethinkdb/node_modules/passport-github/node_modules/passport-oauth/lib/passport-oauth/strategies/oauth2.js:66
  if (!options.clientID) throw new Error('OAuth2Strategy requires a clientID o
                               ^
Error: OAuth2Strategy requires a clientID option
    at Strategy.OAuth2Strategy (/Users/jeffwinkler/rep