-
Notifications
You must be signed in to change notification settings - Fork 7
Add to DefinitelyTyped #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
No, what do I need to do for that? |
I think it would be a great idea to submit indeed ! And I've never published a type, but I think you just need to make a PR here : https://github.com/DefinitelyTyped/DefinitelyTyped It's explained in the readme. |
Yes it would be great as you are the only SSE typedef I could find as well... surprised SSE is so hidden (I know IE is the only browser NOT supporting it, maybe why MSFT doesn't include in their DOM definition either (can't do 'if (window.EventSource) {...use EventSource} else { use long polling/polyfill} I have made a few more tweaks as I'm starting to use it, which I can push back, like: interface IOnMessageEvent extends Event { |
Have you submitted this to DefinitelyTyped?
This file looks good to me.
It just needs a test file.
I'll help if you like...
The text was updated successfully, but these errors were encountered: