balderdashy / sails.io.js

Browser SDK for communicating w/ Sails via sockets
https://sailsjs.com/documentation/reference/web-sockets/socket-client
183 stars 118 forks source link

Request Queue merging from tmp socket to socket #31

Closed JetFault closed 9 years ago

JetFault commented 10 years ago

This is a fix for #19 and pulls in https://github.com/albertosouza/sails.io.js from @albertosouza

mikermcneil commented 9 years ago

Hey @JetFault I'm not seeing this on my end- would you write a failing test for the current master branch stuff? Then we'll be able to see how this PR makes it pass (would make my life much easier). Thanks!

mikermcneil commented 9 years ago

@JetFault I'm working on this while I'm in here doing the sio1.0 upgrade, so let me know if you still see it and we'll figure it out (here's the branch I'm on https://github.com/balderdashy/sails.io.js/tree/new-sio)

dzcpy commented 9 years ago

What's the status about this issue? Is it fixed?