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
Add Withings support #25154
Add Withings support #25154
Conversation
Did you address my comments from the previous PR? |
Yes. The config flow changes were especially welcomed as it simplified the code. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I didn't realize changing my github email would be so messy. At any rate the PR is updated and addresses the latest feedback. |
The branch isn't clean. It has lots of unrelated commits. Please rebase and clean the branch, or open a new PR from a clean branch. |
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.
I think we can merge after these comments are addressed.
Addressing PR feedback. Cleaning up some static code checks. Fixing bug with saving credentials.
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.
Great!
Reopening #21625
Description:
Adding support for connecting to Withings apis to integrate health data into home assistant.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8816
Example entry for
configuration.yaml
(if applicable):Checklist:
tox
. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
REQUIREMENTS
variable (example).requirements_all.txt
by runningscript/gen_requirements_all.py
..coveragerc
.If the code does not interact with devices: