search

andrew / node-xbox-controller

:video_game: [UNMAINTAINED] Interface for Xbox 360 game controller into Node.js
http://libraries.io/npm/xbox-controller
MIT License
270 stars 49 forks source link

Adding support for multiple devices, serial number constructor and example code #36

Closed gordonturner closed 9 years ago

gordonturner commented 9 years ago

I have been working on multiple device support and think I have a reasonable approach.

The pull request replaces the 'name' with 'serial number' as an optional constructor argument. With multiple devices, it is likely that they will share the same name, but the serial number is unique.

Updated the readme to include the changes and added an example.js.

Let me know if you think the pull request needs enhancements?

(Also, thanks for node-xbox-controller, nice to have something to start from!)

andrew commented 9 years ago

Wonderful work @gordonturner, would you also like a commit bit so you can freely contribute to the project?

gordonturner commented 9 years ago

Sure, that would be great, thanks @andrew.

andrew commented 9 years ago

@gordonturner I've added you as a collaborator now, I'll push a new release to npm on Monday when I get home.

gordonturner commented 9 years ago

Hey @andrew could I bother you to push a new release to npm?

Thanks! G.

andrew commented 9 years ago

Done! https://github.com/andrew/node-xbox-controller/releases/tag/v0.7.0