-
Notifications
You must be signed in to change notification settings - Fork 7.7k
feat(docs): Add Zigbee library API documentation #11525
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
base: master
Are you sure you want to change the base?
Conversation
👋 Hello P-R-O-C-H-Y, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
Thank you very much for your contribution, @P-R-O-C-H-Y. It will take some time, but I'll review ASAP. |
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 there are some leftover python and temp files
@lucasssvaz Removed, thanks :) |
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.
Pull Request Overview
This PR adds comprehensive Zigbee library API documentation by introducing new RST files for core and endpoint classes and updating the main libraries index.
- Added
ZigbeeEP
endpoint base class documentation with detailed API breakdown. - Added
ZigbeeCore
main class documentation covering network management and utilities. - Created individual endpoint docs (
ep_*
) for each Zigbee device type and updatedlibraries.rst
TOC.
Reviewed Changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
docs/en/zigbee/zigbee_ep.rst | Added base endpoint API reference |
docs/en/zigbee/zigbee_core.rst | Added core library API reference |
docs/en/zigbee/zigbee.rst | Added overview, topology, and toctree |
docs/en/zigbee/ep_*.rst | Added specific endpoint class docs |
docs/en/libraries.rst | Updated Table of Contents with Zigbee |
Comments suppressed due to low confidence (2)
docs/en/zigbee/zigbee_ep.rst:346
- [nitpick] The description for the callback parameter 'time' is listed under the method instead of alongside the callback signature; consider clarifying its placement under the callback details.
* ``time`` - Identify time in seconds
docs/en/zigbee/ep_door_window_handle.rst:95
- The example section is marked 'To be added'. Please provide a concrete example or remove the placeholder.
*To be added*
* **Router**: Extends network range, routes messages, mains powered devices (typicaly lights, switches, etc.) | ||
* **End Device**: Battery-powered devices that can sleep for extended periods (typicaly sensors) |
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.
Spelling mistake: 'typicaly' should be 'typically'.
* **Router**: Extends network range, routes messages, mains powered devices (typicaly lights, switches, etc.) | |
* **End Device**: Battery-powered devices that can sleep for extended periods (typicaly sensors) | |
* **Router**: Extends network range, routes messages, mains powered devices (typically lights, switches, etc.) | |
* **End Device**: Battery-powered devices that can sleep for extended periods (typically sensors) |
Copilot uses AI. Check for mistakes.
* **Router**: Extends network range, routes messages, mains powered devices (typicaly lights, switches, etc.) | ||
* **End Device**: Battery-powered devices that can sleep for extended periods (typicaly sensors) |
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.
Spelling mistake: 'typicaly' should be 'typically'.
* **Router**: Extends network range, routes messages, mains powered devices (typicaly lights, switches, etc.) | |
* **End Device**: Battery-powered devices that can sleep for extended periods (typicaly sensors) | |
* **Router**: Extends network range, routes messages, mains powered devices (typically lights, switches, etc.) | |
* **End Device**: Battery-powered devices that can sleep for extended periods (typically sensors) |
Copilot uses AI. Check for mistakes.
********** | ||
|
||
For better debugging, you can enable debug mode to get detailed information about network operations. Call debug mode before starting Zigbee. | ||
Also selectiong zigbee mode with *debug* suffix is recommended. |
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.
Spelling mistake: 'selectiong' should be 'selecting'.
Also selectiong zigbee mode with *debug* suffix is recommended. | |
Also selecting zigbee mode with *debug* suffix is recommended. |
Copilot uses AI. Check for mistakes.
Description of Change
This pull request adds documentation for new Zigbee library, expanding the API coverage.
libraries.rst
that organizes Zigbee-related APIs under their own subsection, improving navigation and discoverability.Tests scenarios
Related links
Closes #10135