search

balderdashy / sails

Realtime MVC Framework for Node.js
https://sailsjs.com
MIT License
22.82k stars 1.95k forks source link

sails.sockets.addRoomMembersToRooms is not a function (0.12.3) #3694

Closed NOaMTL closed 8 years ago

NOaMTL commented 8 years ago

Sails version: 0.12.3 Node version: 4.4.2 NPM version: 3.8.7 Operating system: Win7 64

I had this issue #3558 and updated my sails from 0.12.0 to 0.12.3 and now I have this issue :

C:\Users\Yohan\AppData\Roaming\npm\node_modules\sails\lib\hooks\pubsub\index.js:1342 sails.sockets.addRoomMembersToRooms(self._classRoom(), rooms );

this happends when I use the model.publishCreate(). (I am not using sails.sockets.addRoomMembersToRooms in my controller)

sailsbot commented 8 years ago

Hi @NOaMTL! It looks like you missed a step or two when you created your issue. Please edit your comment (use the pencil icon at the top-right corner of the comment box) and fix the following:

As soon as those items are rectified, post a new comment (e.g. “Ok, fixed!”) below and we'll take a look. Thanks!

If you feel this message is in error, or you want to debate the merits of my existence (sniffle), please contact inquiries@treeline.io.

NOaMTL commented 8 years ago

Ok, fixed! (typo)

mikermcneil commented 8 years ago

Hi @NOaMTL, would you mind checking the version in node_modules/sails/node_modules/sails-hook-sockets/package.json (or the NPM 3 equivalent)? Thanks!

sailsbot commented 8 years ago

@NOaMTL,@sailsbot,@mikermcneil: Hello, I'm a repo bot-- nice to meet you!

It has been 30 days since there have been any updates or new comments on this page. If this issue has been resolved, feel free to disregard the rest of this message and simply close the issue if possible. On the other hand, if you are still waiting on a patch, please post a comment to keep the thread alive (with any new information you can provide).

If no further activity occurs on this thread within the next 3 days, the issue will automatically be closed.

Thanks so much for your help!