Skip to content

Updated documentation for SABnzbd.#4836

Merged
frenck merged 6 commits into
home-assistant:nextfrom
jeradM:sabnzbd
May 16, 2018
Merged

Updated documentation for SABnzbd.#4836
frenck merged 6 commits into
home-assistant:nextfrom
jeradM:sabnzbd

Conversation

@jeradM
Copy link
Copy Markdown
Member

@jeradM jeradM commented Mar 5, 2018

Description:
Updated documentation for sensor.sabnzbd to reflect that it has been moved to a component with additional configuration options and services.

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

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.

@frenck frenck added the new-integration This PR adds documentation for a new Home Assistant integration label Mar 15, 2018
@frenck frenck self-assigned this Mar 23, 2018
@frenck
Copy link
Copy Markdown
Member

frenck commented Mar 23, 2018

Rebased the PR in order to fix the build

frenck
frenck previously approved these changes Mar 23, 2018
@frenck
Copy link
Copy Markdown
Member

frenck commented Mar 23, 2018

Thanks, @jeradM! 🥇

Can be merged as soon as the parent PR gets merged.

@frenck frenck removed their assignment Mar 23, 2018
@frenck frenck assigned frenck and jeradM and unassigned frenck Apr 6, 2018
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Hey @jeradM,
Sorry m8, but some merged conflict appeared out of nowhere.
Could you please take a look and resolve those? 👍

@jeradM
Copy link
Copy Markdown
Member Author

jeradM commented Apr 7, 2018

Yep no problem. It looks like #4942 modified the old documentation

@jeradM
Copy link
Copy Markdown
Member Author

jeradM commented Apr 7, 2018

Rebased next branch. Merge conflicts should be resolved now

@MartinHjelmare
Copy link
Copy Markdown
Member

MartinHjelmare commented May 7, 2018

There is still a merge conflict. The sensor page should not be removed, just streamlined and link to the component page, saying it's required to set up sensors.

@jeradM
Copy link
Copy Markdown
Member Author

jeradM commented May 7, 2018

Ah it was a new merge conflict from #5230. Should be good to go now. I updated ha_release to 0.69, but I'm wondering, since this is a breaking change, should it wait until 0.70 so it makes it into an early beta?

@MartinHjelmare
Copy link
Copy Markdown
Member

Yeah, this will not go out in 0.69 but 0.70.

@frenck frenck assigned frenck and unassigned jeradM May 15, 2018
@frenck
Copy link
Copy Markdown
Member

frenck commented May 16, 2018

Thank you for this @jeradM.
I'll go ahead and merge this one in.

@frenck frenck merged commit 9297150 into home-assistant:next May 16, 2018
@jeradM jeradM deleted the sabnzbd branch May 17, 2018 02:08
Oro pushed a commit to Oro/home-assistant.github.io that referenced this pull request May 20, 2018
* Updated documentation for SABnzbd.

* Added a period

* ✏️ Tweaks

* ⬆️ ha_release -> 0.68

* Keep sensor page with link to component. Fix merge conflict from home-assistant#5230

* ⬆️ ha_release -> 0.70
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-integration This PR adds documentation for a new Home Assistant integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants