teslamate icon indicating copy to clipboard operation
teslamate copied to clipboard

[WIP] typo docs/home_assistant script.yaml->scripts.yaml

Open baylanger opened this issue 4 years ago • 5 comments

While doing my own setup, I noticed this slight typo.

baylanger avatar Jun 19 '21 12:06 baylanger

@mnadvornik fyi.

baylanger avatar Jun 19 '21 12:06 baylanger

I just realized this might be needed as well with automation.yaml but maybe I'm doing something wrong.

The default configuration.yaml comes with :

automation: !include automations.yaml
script: !include scripts.yaml

Perhaps those are used by the system and it's recommended for "custom" config (automation, script, etc) to use a different file... I'll do some googling a little later.

I'm still quite knew to HA, still learning.

baylanger avatar Jun 19 '21 13:06 baylanger

If I set the following in configuration.yaml

automation: !include automations.yaml
script: !include scripts.yaml
automation: !include automation.yaml
script: !include script.yaml

This shows up :

2021-06-19 09:04:41 WARNING (SyncWorker_0) [homeassistant.util.yaml.loader] YAML file /config/configuration.yaml contains duplicate key "automation". Check lines 8 and 66
2021-06-19 09:04:41 WARNING (SyncWorker_0) [homeassistant.util.yaml.loader] YAML file /config/configuration.yaml contains duplicate key "script". Check lines 9 and 67

Based on ha-docs

I ended up doing this :

tesla automation: !include tesla-automation.yaml
tesla script: !include tesla-script.yaml

Maybe it makes sense to tag those with "tesla" prefix?

baylanger avatar Jun 19 '21 13:06 baylanger

If you copy the automation script, you get: Message malformed: expected dictionary

ColoradoBills avatar Jun 25 '21 23:06 ColoradoBills

You cant have duplicated automation or script in your configuration.

Use

automation: !include_dir_merge_list ./automation

then just create a folder automation snd make a new file teslamate.yaml with all your automations.

kimmilde avatar Jun 26 '21 15:06 kimmilde

I am going to close this. If we need to review again, lets open a new PR as the script.yaml isn't even added to the configuration.yaml. We probably could just add script(s).yaml to the configuatiuon.yaml. I think there has been changes since this was created so if anything, it probably needs different changes.

https://docs.teslamate.org/docs/integrations/home_assistant#configurationyaml

Dulanic avatar Nov 13 '23 17:11 Dulanic