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.

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

Refactor for public release with latest integration library #13

Merged
merged 29 commits into from
Mar 6, 2024
Merged

Refactor for public release with latest integration library #13

merged 29 commits into from
Mar 6, 2024

Conversation

zehnm
Copy link
Contributor

@zehnm zehnm commented Mar 6, 2024

This PR is a major rewrite to support more features and to release the project as open source.

Added

Fixed

  • Only discover Apple TV devices and no longer HomePods (feature-and-bug-tracker#173).
  • Improved disconnect & reconnect handling. This should prevent the reported BlockedStateError issues (feature-and-bug-tracker#300).
  • Automatically wake up Apple TV if a command is sent while it is in standby.

Changed

  • Play/Pause will exit screensaver and continue playing paused media (tested with Apple TV+, YouTube).
  • Updated pyatv client library to latest 0.14.5 release for common bug fixes and improvements.

zehnm added 29 commits February 28, 2024 14:30
@zehnm
WIP (api events broken) project refactoring
f72d92e 
- use public Python integration library 0.1.3
- use Python naming conventions
- separate driver into config & discovery modules
- ci build uses pyinstaller docker build image
@zehnm
WIP refactor: own module for driver setup flow
f682378
@zehnm
build: ucapi 0.1.4
aef1a80
@zehnm
ci: enable automated builds on push
af5d53c
@zehnm
WIP Cleaning up AppleTv class
8d0b3ef 
- Exception handler wrapper for handling pyatv errors
- missing docstrings
- type parameters
@zehnm
build: add IntelliJ project configuration
8134668
@zehnm
build: ucapi 0.1.5
2aadfe3
@zehnm
fixup! ci: enable automated builds on push
187635c
@zehnm
WIP Cleaning up driver.py
1583298
@zehnm
fixup! ci: enable automated builds on push
5a56826
@zehnm
WIP fix all linting issues
d090e40
@zehnm
Allow manual IP address to setup AppleTV
4a52710 
Improved translations.
@zehnm
Improved error handling for invalid configuration file
70d050c
@zehnm
WIP working setup flow & basic functionality
f6ff478
@zehnm
fix update event, add new key commands
6ddd86e
@zehnm
test new key commands with color buttons
097501b
@zehnm
WIP bug fixes, clean up, new commands
3718dee
@zehnm
Update ucapi with new media-player features
11b8240
@zehnm
Enhance device configuration with address field
ecf0c08
@zehnm
AppleTv class cleanup
f845b92 
- No public fields
- Use enum for ATV protocols
- Reconnect after disconnect notification
- Handle BlockedStateError
@zehnm
AppleTv class cleanup & manual device setup
213950e 
Setup ATV with manual IP address
@zehnm
fix: only discover Apple TV devices
c808c3a 
feature-and-bug-tracker#173
@zehnm
Initial changelog, clean up
0aebc06
@zehnm
Bug fixes, enhanced setup flow, initial multi-device support
200d667
@zehnm
Connect ATV when subscribing to entities
308f7f6 
This should prevent the device state being stuck in UNAVAILABLE after setup.
@zehnm
Log device name in AppleTv
a737780
@zehnm
Fix driver state handling with multiple devices
422e645 
Paired devices are not automatically set as a configured device.
Wait with device connection until subscribe_events request from R2.
@zehnm
Power and device state handling fixes
404ea04 
- detect active screensaver
- wakeup device if command is received
- limit entity updates if data hasn't changed
- reduce device polling to 10s
@zehnm
Update docs
f1994cd
@zehnm zehnm requested a review from martonborzak March 6, 2024 21:24
@zehnm zehnm merged commit f43f802 into main Mar 6, 2024
5 checks passed
@zehnm zehnm deleted the refactor/intg-lib-0.1.3 branch March 6, 2024 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martonborzak martonborzak martonborzak approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support multiple Apple TV devices Missing setup instruction translations
2 participants
@zehnm @martonborzak