SocketIO4Net.Client thoughts

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

Thoughts:

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

 

Update.

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

Coordinator
Jan 24, 2012 at 11:47 AM

torque -

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

Thanks,

Jim

 

 

Jan 24, 2012 at 5: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 3:29 PM

I agree about changing Client for SocketIOClient.

Thanks for putting this solution together.

Great work.