Skip to content

is_allowed_path: Also unit test folder #12810

Merged
pvizeli merged 1 commit intohome-assistant:devfrom
kellerza:path
Mar 1, 2018
Merged

is_allowed_path: Also unit test folder #12810
pvizeli merged 1 commit intohome-assistant:devfrom
kellerza:path

Conversation

@kellerza
Copy link
Copy Markdown
Member

@kellerza kellerza commented Mar 1, 2018

Description:

Fix is_allowed_path and unit test

Related issue (if applicable): fixes #12788 #12807

Checklist:

  • The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

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.

@kellerza kellerza requested a review from a team as a code owner March 1, 2018 07:42
@homeassistant homeassistant added core small-pr PRs with less than 30 lines. cla-signed labels Mar 1, 2018
@kellerza kellerza added this to the 0.64.2 milestone Mar 1, 2018
@pvizeli pvizeli merged commit ff83efe into home-assistant:dev Mar 1, 2018
@Pharizna
Copy link
Copy Markdown

Pharizna commented Mar 1, 2018

I’ve just installed the 0.64.2 but the problem remains :(

ed453ffd-537c-4346-8d97-7007fbb48585

@kellerza kellerza deleted the path branch March 1, 2018 21:34
@kellerza
Copy link
Copy Markdown
Member Author

kellerza commented Mar 1, 2018

@Pharizna it was too late for #12808 0.64.2

@kellerza kellerza modified the milestones: 0.64.2, 0.64.3 Mar 1, 2018
@balloob balloob mentioned this pull request Mar 2, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Jul 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed core small-pr PRs with less than 30 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problems with the newest FOLDER component

4 participants