-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Update TP-Link Smart Home
integration docs - add new supported camera
#38691
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
Conversation
- Add 'C-120' to supported camera models - Personally tested and worked right out of the box with Camera User/PW enabled and "third party compatibility" enabled.
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
tplink smarthome
integration docs - add new supported camera
tplink smarthome
integration docs - add new supported cameraTP-Link Smart Home
integration docs - add new supported camera
📝 WalkthroughWalkthroughThe change updates documentation to include the "C120" camera model in the list of supported Tapo camera devices for the TP-Link integration. The C120 is added between the C100 and C210 models in the enumerated list. No code, configuration, or functional logic was altered; the update is limited solely to documentation content. Changes
Sequence Diagram(s)No sequence diagram is generated as the change is strictly documentation-related. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
⏰ Context from checks skipped due to timeout of 90000ms (3)
🔇 Additional comments (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Thank you, @ndom91 👍
@ndom91 if you don't mind, please consider contributing a fixture file upstream as described in https://python-kasa.readthedocs.io/en/stable/contribute.html#contributing-fixture-files . The upstream project maintains a list of supported devices, and we usually sync it with the home assistant docs when we create a new release, so doing so would help to keep the lists in sync :-) |
Proposed change
C120
to supported camera modelsType of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit