Skip to content

Create sensor.tof #8672

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

Merged
merged 16 commits into from
Mar 11, 2019
Merged

Create sensor.tof #8672

merged 16 commits into from
Mar 11, 2019

Conversation

josemotta
Copy link
Contributor

@josemotta josemotta commented Feb 21, 2019

Description:

Pull request in home-assistant: home-assistant/core#21230

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follows the standards.

@ghost ghost added the to-do label Feb 21, 2019
@klaasnicolaas klaasnicolaas added new-integration This PR adds documentation for a new Home Assistant integration ready-for-review This PR needs to be reviewed next This PR goes into the next branch has-parent This PR has a parent PR in a other repo and removed to-do labels Feb 21, 2019
Co-Authored-By: josemotta <josemotta@bampli.com>
Co-Authored-By: josemotta <josemotta@bampli.com>
@josemotta
Copy link
Contributor Author

Having some issues also at pull request at home assistant. Any help?

Pull request in home-assistant: home-assistant/home-assistant#21230

@josemotta josemotta closed this Feb 22, 2019
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Feb 22, 2019
@klaasnicolaas
Copy link
Member

Why closing this PR?

@josemotta
Copy link
Contributor Author

josemotta commented Feb 22, 2019

Sorry, wrong action. Please keep it! It´s my first time here I was trying to ask for help at code PR;

Thanks for your support

@josemotta
Copy link
Contributor Author

reopening

@josemotta josemotta reopened this Feb 22, 2019
@ghost ghost added the to-do label Feb 22, 2019
@klaasnicolaas klaasnicolaas added ready-for-review This PR needs to be reviewed and removed to-do labels Feb 22, 2019
@josemotta
Copy link
Contributor Author

josemotta commented Feb 26, 2019

There is a broken link at top right:

Source: tof/sensor.py

Actually it should be sensor/tof.py, isn´t? How to fix this?

https://deploy-preview-8672--home-assistant-docs.netlify.com/components/sensor.tof/

@frenck
Copy link
Member

frenck commented Feb 27, 2019

This is known and has to do with "The Great Migration".

The documentation code currently only looks at the "new" situation.

From the documentation perspective, there is currently nothing you can do about that.

@frenck frenck added the parent-merged The parent PR has been merged already label Mar 8, 2019
MartinHjelmare
MartinHjelmare previously approved these changes Mar 8, 2019
frenck
frenck previously approved these changes Mar 11, 2019
Signed-off-by: Franck Nijhof <frenck@addons.community>
@frenck
Copy link
Member

frenck commented Mar 11, 2019

Thanks, @josemotta! 👍

@frenck frenck merged commit 9a505f2 into home-assistant:next Mar 11, 2019
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Mar 11, 2019
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 a other repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch parent-merged The parent PR has been merged already
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants