asosnovsky / Shortumation

[Not Maintained] A better way to write automations in home-assistant without having to install and run a seperate automation engine!
MIT License
302 stars 7 forks source link

Error if Description is missing in automation #160

Closed ghost closed 1 year ago

ghost commented 1 year ago

Describe the bug I have added Shortmation as a panel_iframe so it appears in the sidebar. When I select it, I get a popup over the main screen entitled "Invalid Automations detected!"

After sone testing, I can confirm this error only appears in automations that do not have the Description: line in them. This is optional of you are writing your automations in yaml:

To Reproduce Steps to reproduce the behavior:

  1. Remove Description: from any yaml automation
  2. Click on Shortumation

Or even paste your automation file here Expected behavior I would expect Shortumation to handle this is an optional line in a manually written automation.

Screenshots Screenshot 2022-09-02 at 09 12 19

Desktop (please complete the following information): n/a

Smartphone (please complete the following information): n/a

Additional context

Add any other context about the problem here. The workaround is to add Description: '' to each automation. I can confirm that having added this line to every automation, I have no errors 👍

Shortumation Logs Added as a file as report to ling to post otherwise. logs.log

asosnovsky commented 1 year ago

@jchhenderson this should be out in the next version v0.7.3, I misconfigured what "None" is in the backend vs "null" / "undefined" in the frontend :facepalm:

fyi, just wondering, since you are using the docker container version, how do you get notified when a newer version is out? just by looking at this repo?

ghost commented 1 year ago

this should be out in the next version v0.7.3

great!

how do you get notified when a newer version is out? just by looking at this repo?

Correct. I watch this repo,.