search

scttnlsn / backbone.io

Backbone.js sync via Socket.IO
http://scttnlsn.github.io/backbone.io
540 stars 66 forks source link

bug when no channel is used with backend #13

Closed jhpinson closed 12 years ago

jhpinson commented 12 years ago

when using RedisStore with socket.io

scttnlsn commented 12 years ago

Seems like something the Redis store should be handling. It might make more sense anyway to conditionally socket.set('channel', ...) only if a channel was specified- no need to store a null value.

jhpinson commented 12 years ago

totally agree, i let you fix the bug

scttnlsn commented 12 years ago

Pushed the change. Care to test it out with the Redis store?