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.

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

Update yahooweather 0.8 / change request time #3352

Merged
merged 1 commit into from
Sep 14, 2016

Conversation

pvizeli
Copy link
Member

@pvizeli pvizeli commented Sep 12, 2016

Description:

  • Update yahooweather to 0.8 to reduce error in logs
  • Add CONF_NAME support
  • Rename min/max temperature value
  • Change update time to 10 minute, so 2 minute are to small and give a lot of false request

Bad change:

  • Min/Max temperature change name from Temperature to Temperature min and Temperature max for better understand.
  • Default name change from weather -> yweather

Pull request in home-assistant.io with documentation (if applicable): This version is official in docu bot not supported at the moment from platform... This PR change that

Checklist:

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

@balloob
Copy link
Member

balloob commented Sep 14, 2016

Looks good! 🐬

This is a breaking change and should be noted as such in release notes

@balloob balloob merged commit b78e987 into home-assistant:dev Sep 14, 2016
hartmms pushed a commit to hartmms/home-assistant that referenced this pull request Sep 17, 2016
@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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants