Skip to content

(Ignore) Add STATE_AUTO support to generic_thermostat#5898

Closed
niklaswa wants to merge 102 commits intohome-assistant:nextfrom
niklaswa:patch-3
Closed

(Ignore) Add STATE_AUTO support to generic_thermostat#5898
niklaswa wants to merge 102 commits intohome-assistant:nextfrom
niklaswa:patch-3

Conversation

@niklaswa
Copy link
Copy Markdown
Contributor

Description:
Make the user aware of what STATE_AUTO does.

Pull request in home-assistant (if applicable): home-assistant/core#15678

Checklist:

  • Branch: Fixes, changes and adjustments should be created against current. New documentation for platforms/components and features should go to next.
  • The documentation follow the standards.

balloob and others added 30 commits July 16, 2018 09:01
* Updates for future beta

* Update views changelog

* Updated based on PR feedback
outdated config - corrected
seems simple once you know, but placing a reference here is certainly helpful for new folks.
* Add Tuya light document

* fix grammar problem
Use the example code actually not working due to the Beijing US Embassy station changed it's name.
@DubhAd DubhAd added the has-parent This PR has a parent PR in another repo label Jul 28, 2018
@frenck frenck added ready-for-review This PR needs to be reviewed needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch in-progress This PR/Issue is currently being worked on and removed to-do ready-for-review This PR needs to be reviewed labels Jul 29, 2018
@frenck
Copy link
Copy Markdown
Member

frenck commented Jul 29, 2018

@niklaswa, thank you for this, nevertheless, the PR is targeted against the wrong branch.
Since this relies on a featured pending a parent PR in our code base, the documentation should go into the next branch. Please rebase.

@niklaswa niklaswa changed the base branch from current to next July 29, 2018 07:55
@niklaswa
Copy link
Copy Markdown
Contributor Author

niklaswa commented Jul 29, 2018

Done. Looks weird but I hope that was right? 👀 @frenck

@frenck
Copy link
Copy Markdown
Member

frenck commented Jul 29, 2018

Yeah, it does, you should have rebased instead of changing the target in GitHub and merge in next.
Because of your actions, probably the only solution now is to close this PR and create a new one.

@niklaswa
Copy link
Copy Markdown
Contributor Author

New one #5916 @frenck

@niklaswa niklaswa closed this Jul 29, 2018
@ghost ghost removed the in-progress This PR/Issue is currently being worked on label Jul 29, 2018
@niklaswa niklaswa changed the title Add STATE_AUTO support to generic_thermostat (Ignore) Add STATE_AUTO support to generic_thermostat Jul 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.