-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Added "Waze travel time" sensor documentation #4498
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
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 for the PR @Myrddyn1.
I've reviewed it and left some comments, could you please take a look? 👍
.rake_tasks~
Outdated
@@ -0,0 +1,19 @@ | |||
clean |
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.
This file should not be here?
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 deleted the file
logo: waze.png | ||
ha_category: Transport | ||
ha_iot_class: "Cloud Polling" | ||
ha_release: 0.61 |
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.
0.61
is already shipped some time ago. Please set it to 0.63
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.
Done
- route | ||
``` | ||
|
||
Configuration variables: |
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.
Please use the configuration tags, for more information please see:
https://home-assistant.io/developers/documentation/create_page/#configuration
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've made the requested changes using the new tags
- **destination** (*Required*): Enter the destination address or the GPS coordinates of the location (GPS coordinates has to be separated by a comma). | ||
- **region** (*Required*): Choose one of the available regions from 'EU', 'US', 'NA' (equivalent to 'US'), 'IL'. Default value is 'US'. | ||
- **outputs** (*Required*): At least one output type (see below) is required. | ||
- *duration*: The sensor will display the duration of the best chosen route, in min. |
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.
best chosen
->best-chosen
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.
Corrected
- **region** (*Required*): Choose one of the available regions from 'EU', 'US', 'NA' (equivalent to 'US'), 'IL'. Default value is 'US'. | ||
- **outputs** (*Required*): At least one output type (see below) is required. | ||
- *duration*: The sensor will display the duration of the best chosen route, in min. | ||
- *distance*: The sensor will display the distance of the best chosen route, in km. |
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.
best chosen
->best-chosen
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.
Corrected
- **outputs** (*Required*): At least one output type (see below) is required. | ||
- *duration*: The sensor will display the duration of the best chosen route, in min. | ||
- *distance*: The sensor will display the distance of the best chosen route, in km. | ||
- *route*: The sensor will display the main steps of the best chosen route. |
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.
best chosen
->best-chosen
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.
Corrected
logo: waze.png | ||
ha_category: Transport | ||
ha_iot_class: "Cloud Polling" | ||
ha_release: 0.63 |
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.
0.63
has been release some time ago. Please set it to the next version.
Fixed commit remarks from frenck
Fixed commit remarks from frenck
Rebased the PR in order to fix the build |
Thanks, @Myrddyn1! 🎖 Can be merged as soon as the parent PR gets merged. |
Description:
Documentation for Waze Travel Time sensor.
Pull request in home-assistant: home-assistant/core#11870