-
-
Notifications
You must be signed in to change notification settings - Fork 100
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
Support arrow keys for ATV4 #59
Conversation
@dale3h Could you maybe help me out and verify this? 😄 I would like to get it in for 0.3.0 so I can add additional controls to Home Assistant. |
Yes, I will try this once I return home this week. |
@dale3h Did you ever get around to try this out? 😄 |
No, I apologize, I have not. To be honest, I completely forgot! I will set myself a location-based reminder so that it alerts me anytime I get home, that way I can make sure that I get it tested. |
It doesn't seem to work when using
|
cd6ac22
to
2d687db
Compare
@dale3h I realized I did some stuff wrong and have now corrected issues I found. Would be great if you could re-test my latest push 😄 |
@msiedlarek Great, thanks for testing 😄 I'm gonna try to get this in ASAP. |
2d687db
to
d7624e6
Compare
This fixes #17.
d7624e6
to
d0993d8
Compare
I tried this with my ATV3 now and it works as well, which is nice! 👍 |
This is the quick-n-easy solution where I ignore the fact that ATV3 does not seem to support these commands. But since it just ignores them I find it to be good enough since it does not increase complexity of handling multiple device types in the library.
I have not verified if this PR works since I don't own a gen 4 device, so if someone can help me with that I would be grateful. Will not merge until verification is done.