add data_grandlyon documentation#44658
add data_grandlyon documentation#44658Crocmagnon wants to merge 1 commit intohome-assistant:nextfrom
Conversation
|
It seems that this PR is targeted against an incorrect branch since it has a parent PR on one of our codebases. Documentation that needs to be updated for an upcoming release should target the |
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
085cc14 to
797ee82
Compare
c6b0512 to
82320e2
Compare
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds documentation for the new data_grandlyon Home Assistant integration, covering prerequisites, configuration flow, entities, and troubleshooting.
Changes:
- Added a new integration documentation page for Data Grand Lyon.
- Documented config flow fields and sub-entry setup for transit stops and Vélo'v stations.
- Described entities, update interval, limitations, and common troubleshooting steps.
| required: inclusive | ||
| type: string | ||
| Password: | ||
| description: "Your password on data.grandlyon.com. Required if you want to monitor transit stops." | ||
| required: inclusive |
There was a problem hiding this comment.
required: inclusive is likely not a valid value for configuration_basic fields (the docs schema typically expects true/false). If these credentials are only needed for transit stops, set them as not required in the table and keep the conditional requirement in the description, or split the documentation to clearly indicate when credentials are required.
| required: inclusive | |
| type: string | |
| Password: | |
| description: "Your password on data.grandlyon.com. Required if you want to monitor transit stops." | |
| required: inclusive | |
| required: false | |
| type: string | |
| Password: | |
| description: "Your password on data.grandlyon.com. Required if you want to monitor transit stops." | |
| required: false |
There was a problem hiding this comment.
unclear whether this is true or not.
the docs mention required: inclusive
82320e2 to
78d8ce2
Compare
78d8ce2 to
565ef92
Compare
Proposed change
Type of change
currentbranch).currentbranch).nextbranch).nextbranch).Additional information
Checklist
currentbranch.nextbranch.