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.

Sign up for GitHub

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

Jump to bottom

DO NOT MERGE - Exploring TypeScript support #89

Open

NAllred91 wants to merge 2 commits into OpenXbox:develop from NAllred91:develop

NAllred91 commented Apr 26, 2022

Creating this PR to discuss migrating to TypeScript.

The API surface in this commit is slightly different than the current implementation. If backwards compatibility is a concern this can be adjusted.

Some things that are still incomplete:

Other channels besides the Input Channel.
Starting/Stopping channels.
Authenticated connections.
Need more tests.

Some things that are improved:

Connection should be more stable.
Less dgram usage. (One discovery socket, one dedicated socket per xbox)
Optional auto reconnect.
Input channel continuously streams gamepad state. (I think I still want to slow down when nothing is happening on the gamepad)

I've included some demos with comments that should be fairly self explanatory.

Looking forward to your feedback!


          Initial commit exploring TypeScript support

9d45491

NAllred91 changed the title ~~DO NOT MERGE - Initial commit exploring TypeScript support~~ DO NOT MERGE - Exploring TypeScript support


          Cleaning up comments

2cec29d

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

Labels

None yet