Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use voluptuous for Telldus Live #3292

Merged
merged 1 commit into from
Sep 11, 2016
Merged

Use voluptuous for Telldus Live #3292

merged 1 commit into from
Sep 11, 2016

Conversation

fabaff
Copy link
Member

@fabaff fabaff commented Sep 9, 2016

Description:
Migration of the configuration check to voluptuous.

Related issue (if applicable): fixes 127528299

Example entry for configuration.yaml (if applicable):

switch:
tellduslive:
  public_key: ABCDEFGHJKLMNOPQRSTUVXYZ
  private_key: ABCDEFGHJKLMNOPQRSTUVXYZ
  token: ABCDEFGHJKLMNOPQRSTUVXYZ
  token_secret: ABCDEFGHJKLMNOPQRSTUVXYZ

@molobrakos , would be nice if you could take a look at the changes and run a quick test. Thanks.

@fabaff fabaff mentioned this pull request Sep 9, 2016
Closed
@balloob
Copy link
Member

balloob commented Sep 10, 2016

🐬

@fabaff fabaff merged commit ac9151a into home-assistant:dev Sep 11, 2016
@fabaff fabaff deleted the voluptuous-telldus-live branch September 11, 2016 07:22
@molobrakos
Copy link
Contributor

Sorry for my late feedback, @fabaff - but it both looks fine and works fine - thank you for your help migrating this component!

@home-assistant home-assistant locked and limited conversation to collaborators Mar 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
voluptuous
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fabaff @balloob @molobrakos