Skip to content

Fix automation failing to setup if no automations specified#7647

Merged
balloob merged 1 commit into
devfrom
fix-empty-automation
May 18, 2017
Merged

Fix automation failing to setup if no automations specified#7647
balloob merged 1 commit into
devfrom
fix-empty-automation

Conversation

@balloob
Copy link
Copy Markdown
Member

@balloob balloob commented May 18, 2017

Description:

The automation component would fail to setup if no automations specified. Since the introduction of the automation editor we enable the automation editor in the default config, causing that to break out of the box. This fixes it by allowing the automation editor to setup correctly when there are no automations.

Example entry for configuration.yaml (if applicable):

automation:

Checklist:

If the code does not interact with devices:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • Tests have been added to verify that the new code works.

@mention-bot
Copy link
Copy Markdown

@balloob, thanks for your PR! By analyzing the history of the files in this pull request, we identified @pvizeli, @fabaff and @philipbl to be potential reviewers.

@balloob balloob merged commit e773133 into dev May 18, 2017
@balloob balloob deleted the fix-empty-automation branch May 18, 2017 04:58
@home-assistant home-assistant locked and limited conversation to collaborators Sep 4, 2017
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.

4 participants