Skip to content

Fix poorly formatted automations#14196

Merged
balloob merged 1 commit intodevfrom
automation-editor-fix
Apr 30, 2018
Merged

Fix poorly formatted automations#14196
balloob merged 1 commit intodevfrom
automation-editor-fix

Conversation

@balloob
Copy link
Copy Markdown
Member

@balloob balloob commented Apr 30, 2018

Description:

Automation editor would be prevented from writing automations if the user had copy pasted some automations over to automations.yaml but forgot to add an ID (as per the instructions).

Now, if we see an automation without an ID, we'll add one.

Example entry for configuration.yaml (if applicable):

config:
automation: !include automations.yaml

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

  • Tests have been added to verify that the new code works.

@balloob balloob added this to the 0.68.1 milestone Apr 30, 2018
@balloob balloob merged commit 853a169 into dev Apr 30, 2018
@balloob balloob deleted the automation-editor-fix branch April 30, 2018 13:56
balloob added a commit that referenced this pull request Apr 30, 2018
@balloob balloob mentioned this pull request Apr 30, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Sep 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants