search

ghcjs / ghcjs-base

base library for GHCJS for JavaScript interaction and marshalling, used by higher level libraries like JSC
MIT License
45 stars 67 forks source link

Change the WebSocket readyState to reflect the right values #63

Closed manyoo closed 7 years ago

manyoo commented 8 years ago

I'm using the WebSocket API and find out that the readyState values doesn't reflect the right values. Maybe there's a reason? But I've made the change locally and it works great for my app. If this is a bug, hope this fix is merged.

luite commented 8 years ago

I suspect I just made a mistake and used the XHR values and didn't remember to check if they needed to be changed for WebSocket. Will look into this soon.

hamishmack commented 7 years ago

Merged #95 instead. It looked basically the same, but used consistent upper/lower case.