Skip to content

Conversation

nordlead2005
Copy link
Contributor

Description:
Added DarkSky forecast documentation, incorporated changes.

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

balloob and others added 30 commits January 15, 2017 09:10
Fix formatting of link to function as a proper link.
* Add docs for migrator script.
* Update release blog and link to influxdb sections.
* Create notify.facebook.markdown

* Update notify.facebook.markdown

* Update notify.facebook.markdown

* Added Facebook logo
Volvo on call website is now found at: device_tracker.volvooncall.markdown
Added some more verbiage as how to properly configure WinSCP to be able to access the folder structure under .homeassistant with the ability to edit the config files using WinSCP. It took me a bit to find that SCP protocol needed to be selected when creating a new session, so I edited the last paragraph to add that requirement and make it easier for first time installers.
The current systemd unit file will provoke an error because the 'ExecPre' line calls a shell built-in function and all systemd Execs need to use an absolute path. the proposed change sets the python environment using the provided 'Environment' calls as used by systemd.
l00mi and others added 15 commits January 17, 2017 08:02
Example config caused hass to fail to start. Corrected and confirmed working.
Updated port information to cover 2016+ TVs that are supported as of this week.

I've removed the similar model TVs that had the same issue as mine (KU6000), that being they were modern ones using a websocket interface on a different port to the old ones.
* Update sensor.usps.markdown

Added more information on Opting In to MY USPS.

* Update sensor.usps.markdown
* WIP: Update eq3btsmart for new features

This is WIP as it requires a revamped backend library, and should not be merged before the changes to the backend are done. See bimbar/bluepy_devices#6 for details.

* Update climate.eq3btsmart.markdown

simplify the documentation, there is no need to pair the device.
Document -1 for brightness.
Former URL is now 404'ing.
@nordlead2005 nordlead2005 changed the title Patch 1 DarkSky Forecast Documentation Jan 18, 2017
@nordlead2005
Copy link
Contributor Author

ok, I really need to find a decent github guide or someone needs to tell me what to do. I deleted the previous pull request because it had hundreds of commits that I don't know why they were there and my attempt to rebase just made it worse. Maybe I should have ignored them, maybe not. Either way, there is now this pull request which only looks marginally better.

I'm hoping someone can either tell me to leave it as is or what to do to make sure both this pull request and the source code one go through.

@Landrash
Copy link
Contributor

@nordlead2005 I recommend you close this PR and open a new clean one.
Easiest way if you feel unfamiliar with git is to clone a new copy of the repo, add your changes to that and open a new PR. A bit sad but often the easiest workaround.

@Landrash Landrash added the new-feature This PR adds documentation for a new Home Assistant feature to an existing integration label Jan 27, 2017
fabaff added a commit that referenced this pull request Feb 13, 2017
@fabaff
Copy link
Member

fabaff commented Feb 13, 2017

Included with c362cab

@fabaff fabaff closed this Feb 13, 2017
@nordlead2005 nordlead2005 deleted the patch-1 branch February 2, 2018 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature This PR adds documentation for a new Home Assistant feature to an existing integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.