Skip to content
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

Added protocol support for newer dash buttons. Added documentation for Raspberry Pi #35

Merged
merged 15 commits into from
Jan 30, 2017

Conversation

AceoStar
Copy link
Contributor

Hey all.

The biggest change here is adding support for manually specifying the listener protocol. This is useful for working with the newer JK29LP button. I also added a few other minor helpers for getting noobs like myself moving in the right direction such as a debug option that skips the request and some handling in case you have an undefined response due to misconfiguration. I also updated the documentation to reflect some of the updates seen via the node-dash-button library.

I hope it helps!

…Added a debug option for testing button presses. Minor documentation updates.
Added details for getting started on Raspberry Pi. Tested on Raspberry Pi 2 running Pixel w/ multiple refreshes to simulate a fresh install.
Formatting and spelling.
More formatting and unsurprisingly more spelling.
Updated with Protip about Wifi. I wish I had thought to try ethernet sooner as it is much more reliable.
Added ability to count button presses for debug purposes.
Merged in Pull request 36
@AceoStar AceoStar changed the title Added protocol support Added protocol support for newer dash buttons. Added documentation for Raspberry Pi Nov 22, 2016
@legotheboss
Copy link

@maddox Can you approve this? Works pretty well and is quite stable...

@maddox
Copy link
Owner

maddox commented Jan 27, 2017

There's conflicts on this branch. @AceoStar can you get them resolved? I'll merge it after.

@AceoStar
Copy link
Contributor Author

I think that should do it.

@AceoStar
Copy link
Contributor Author

@maddox Let me know if you need anything else. Thanks a bunch!

@maddox maddox merged commit 1511584 into maddox:master Jan 30, 2017
@amcelmon-clgx
Copy link

Thank you!

lorenzos added a commit to lorenzos/dasher that referenced this pull request Feb 26, 2017
maddox added a commit that referenced this pull request Feb 27, 2017
Fixed error logging after #35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants