Skip to content

New appliance support: Fridge, Hub, Hood - #141

Open
JanezKr wants to merge 16 commits into
ofalvai:mainfrom
JanezKr:main
Open

New appliance support: Fridge, Hub, Hood#141
JanezKr wants to merge 16 commits into
ofalvai:mainfrom
JanezKr:main

Conversation

@JanezKr

@JanezKr JanezKr commented Jul 3, 2022

Copy link
Copy Markdown
  • Different program options on Oven
  • New appliances support:
    • Fridge
    • Hub
    • Hood (experimental)

@JanezKr

JanezKr commented Nov 18, 2022

Copy link
Copy Markdown
Author

Please Olivér, can you merge my code too. I add few new appliances in last months.

Tanks in advance,

Janez

Comment thread custom_components/candy/client/__init__.py Outdated
Comment thread custom_components/candy/client/model.py
Comment thread custom_components/candy/client/model.py
Comment thread custom_components/candy/client/model.py Outdated
Comment thread custom_components/candy/client/model.py Outdated
Comment thread custom_components/candy/client/model.py Outdated
Comment thread custom_components/candy/client/model.py Outdated
Comment thread custom_components/candy/client/model.py Outdated
Comment thread custom_components/candy/client/model.py Outdated
Comment thread custom_components/candy/client/model.py Outdated
Comment thread custom_components/candy/config_flow.py Outdated
Comment thread custom_components/candy/binary_sensor.py
Comment thread custom_components/candy/binary_sensor.py
Comment thread custom_components/candy/binary_sensor.py
Comment thread custom_components/candy/binary_sensor.py
@ofalvai

ofalvai commented Nov 19, 2022

Copy link
Copy Markdown
Owner

Thank you @JanezKr for the contribution and apologies for the long delay from me.

I had a quick look at the PR and added a few comments about the various new appliances and features, please take a look.

A few general comments:

  • Please revert the changes that wrap long lines and add extra empty lines here and there. It makes really hard to review this PR, and I don't enforce a maximum line length, please don't wrap existing lines into multiple lines.
  • The tests are failing, please take a look at the CI run

change property from  temSet into temp_set
Some fixes on Fridge and Hood.
…er into carbon_filter_clean_needed

renamed grease_filter into grease_filter_clean_needed  and carbon_filter into carbon_filter_clean_needed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants