Skip to content

Add multi contracts support for Hydroquebec#6392

Merged
pvizeli merged 1 commit into
home-assistant:devfrom
titilambert:hydroquebec
Mar 4, 2017
Merged

Add multi contracts support for Hydroquebec#6392
pvizeli merged 1 commit into
home-assistant:devfrom
titilambert:hydroquebec

Conversation

@titilambert
Copy link
Copy Markdown
Contributor

@titilambert titilambert commented Mar 4, 2017

Description:

This update adds the support of hydroquebec accounts with multiple contracts.
Now, we have to add the contract id when using this component.

This is a configuration breaking change from v0.39.x and earlier

Related issue (if applicable): fixes #5408

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.io#2191

Example entry for configuration.yaml (if applicable):

sensor:
  - platform: hydroquebec
    username: MYUSERNAME
    password: MYPASSWORD
    contract: 123456789
    monitored_variables:
     - period_total_bill
     - period_length
     - period_total_days
     - period_mean_daily_bill
     - period_mean_daily_consumption
     - period_total_consumption
     - period_lower_price_consumption
     - period_higher_price_consumption
     - yesterday_total_consumption
     - yesterday_lower_price_consumption
     - yesterday_higher_price_consumption

Checklist:

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

  • Documentation added/updated in home-assistant.github.io
  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New dependencies are only imported inside functions that use them (example).
  • New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

@mention-bot
Copy link
Copy Markdown

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trailing whitespace

@titilambert titilambert changed the title [WIP] Add multi contracts support for Hydroquebec Add multi contracts support for Hydroquebec Mar 4, 2017
@pvizeli pvizeli merged commit aaa0944 into home-assistant:dev Mar 4, 2017
@pvizeli
Copy link
Copy Markdown
Member

pvizeli commented Mar 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.

HydroQuebec (with multiple address maybe ?)

5 participants