This project is read-only.

SocketIO4Net.Client thoughts

Jan 24, 2012 at 9:36 AM
Edited Jan 24, 2012 at 10:00 AM


1. It actually works! Sweet.

2. Rename Client to something less ambiguous like SocketIOClient

3. Sample / Demo on Home page should show retrieval of data from "news" rather than re-emitting

4. Somewhere we need to know if there is anything missing in the implementation (SSL? etc.), i.e., features not supported



I'm so pleased, you even introduced me to actually using NuGet!

Jan 24, 2012 at 12:47 PM

torque -

Thanks for the feedback, I really appreciate it.  As for your thoughts, agree with them all!  I plan to move the code from onto the Sample/Home page.  Sort of silly showing the emit, but I was following the same pattern demo's page.  I'll also update the 'missing' items as well...





Jan 24, 2012 at 6:16 PM

I don't think it is silly showing the Emit at all. I like that.

I also suggest that the event handlers MessageEventArgs and ErrorEventArgs should have more specific names.

Mar 30, 2012 at 4:29 PM

I agree about changing Client for SocketIOClient.

Thanks for putting this solution together.

Great work.