-
-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Bump blinkpy to 0.23.0 #119418
Bump blinkpy to 0.23.0 #119418
Conversation
Hey there @fronzbot, @mkmer, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
addef32
to
37e33eb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for bumping the dependency! However, could you please update the PR description to contain at least one (or multiple) of the following:
- A link to the release notes of this package version, and all versions in between.
- A link to the changelog of this package.
- A link to a Git(Hub) diff/compare view from the current version to the bumped version.
This allows us to review upstream changes, which is needed to decide this change is working as intended and/or if we can include it in, for example, a patch release of Home Assistant.
Additionally, this seems to upgrade to a beta version of the library (considering the version number); we want to avoid using alpha/beta versions in general. What would be the reason to divert from that now?
Thanks already! 👍
../Frenck
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
I updated the Proposed change section accordingly.
Unfortunately, the changelog of the package itself wasn't updated with details:
Also added in first comment, but for reference, here it is again:
Sure, I think this will be relevant for a patch release of Home Assistant due to the authentication issue being solved.
I am not sure why @fronzbot decided to stay with a beta version for now, but for me the new version is working fine during local tests. Without this beta version, the Blink integration can no longer be freshly added/setup in Home Assistant. Existing authentication tokens can continue to be used and keep working, but as soon as you revoke the token via the Blink app and re-authentication/-setup is needed, the bugfixex from the new package version will be required. I would propose to go with the beta version as a short term patch and then switch to (then hopefully available) stable version for the next Home Assistant release. Until the patch is released, for example, I am locked out of Blink from my production Home Assistant instance running on Home Assistant OS, because I removed the old token via the Blink app by mistake. (App GUI is flaky and sometimes performs action against wrong entry there, I actually wanted to remove one of the test tokens and not my prod token, but anyway...) What do you think? Thanks in advance and best regards, |
Because the beta version was released two days ago and I'm waiting for feedback from more than one user to ensure it fixes all issues. I typically wait about a week between beta and final releas. Please wait until a stable release is added before bumping this library version. |
Thanks for the clarification on your process here. Do you think it would make sense to at least bump to the beta version for the Home Assistant release candidate to get more feedback? Another user reporting success with the new version: |
The next release candidate is still weeks out. So looking at the timeline above, that doesn't add up. |
Understood, thanks. |
LGTM, need to take this out of draft to get it in. |
Thanks! I was waiting on the CI to finish, but now it seems like the test failures are not related to this integration or library. |
- Fix authentication due to Blink API changes. - Fix camera battery_voltage and wifi_strength. - Fix clip recording for Blink Mini and Doorbell.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @mback2k 👍
../Frenck
Proposed change
From blinkpy-0.23.0:
From blinkpy-0.23.0b7 and blinkpy-0.23.0b8:
From blinkpy-0.22.7:
Diff between v0.22.6 and v0.23.0 which unfortunately contains a lot of CI/CD related changes:
fronzbot/blinkpy@v0.22.6...v0.23.0 (maybe focus on .py files there)
Type of change
Additional information
https://community.home-assistant.io/t/start-recording-blink-camera-based-on-motion-or-ringing-of-my-ring-doorbell/382208/15?u=mback2k
Checklist
ruff format homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: