Skip to content
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

teslemetry.set_scheduled_charging: Descriptions for time contradict themselves #136069

Open
NoRi2909 opened this issue Jan 20, 2025 · 1 comment

Comments

@NoRi2909
Copy link
Contributor

NoRi2909 commented Jan 20, 2025

The problem

There are two descriptions for the teslemetry.set_scheduled_charging action that contradict themselves regarding the time to set:

"description": "Sets a time at which charging should be completed.",

"description": "Time to start charging.",

So the action description says "completed" while the field description says "start".
I assume it should be "start" or "started" in both places.

Stumbled over this while verifying the German translations.

BTW, to avoid any wrong translation:
I assume that "Time" is a time of day here, not a timespan after which to start the charging.

What version of Home Assistant Core has the issue?

core-2025.1.2

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Teslemetry

Link to integration documentation on our website

https://www.home-assistant.io/integrations/teslemetry/

@home-assistant
Copy link

Hey there @Bre77, mind taking a look at this issue as it has been labeled with an integration (teslemetry) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of teslemetry can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign teslemetry Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


teslemetry documentation
teslemetry source
(message by IssueLinks)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant