Skip to content

Fix for issue #3#6

Open
PatrickG wants to merge 1 commit intoweixiyen:masterfrom
PatrickG:master
Open

Fix for issue #3#6
PatrickG wants to merge 1 commit intoweixiyen:masterfrom
PatrickG:master

Conversation

@PatrickG
Copy link
Copy Markdown

If a request is executed before a connection exists, save this temporarily and send it after a connection has been established.

If a request is executed before a connection exists, save this temporarily and send it after a connection has been established.
@funston
Copy link
Copy Markdown

funston commented Nov 8, 2012

feels like a pretty big fail in the library that it doesn't either:

  1. let the client know when a connect happens
  2. buffer any shout/requests internally in speaker.js until it has

asking us to do a settimeout or other hacks is silly.

@PatrickG
Copy link
Copy Markdown
Author

PatrickG commented Nov 9, 2012

I've found a better module: axon (https://github.com/visionmedia/axon).
I've not tested it yet, but the docs are looking very nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants