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

Add troubleshooting guide for cross-subnet communications #459

Merged
merged 1 commit into from
Jan 20, 2019

Conversation

domibarton
Copy link
Contributor

This troubleshooting guide is the result of the outcome of issue #422.

Please have a look at the issue for more insights:
#422

@domibarton
Copy link
Contributor Author

If @cnrd tells me the name of his device, I can add it to the list before merging.

@coveralls
Copy link

coveralls commented Jan 20, 2019

Coverage Status

Coverage remained the same at 72.461% when pulling 4d33ef2 on confirm:troubleshooting-guide into 6b2ce47 on rytilahti:master.

@cnrd
Copy link

cnrd commented Jan 20, 2019

This is the device: https://www.home-assistant.io/components/remote.xiaomi_miio/ the internal name in various pieces of code is chuangmi_ir.

Copy link
Owner

@rytilahti rytilahti left a comment

Choose a reason for hiding this comment

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

Looks good, thanks! :-) You could add a link to the issue in case someone wants to read more about how you came to these conclusions.

@domibarton
Copy link
Contributor Author

Looks good, thanks! :-) You could add a link to the issue in case someone wants to read more about how you came to these conclusions.

good point

@domibarton domibarton force-pushed the troubleshooting-guide branch from 6e258f2 to 5962903 Compare January 20, 2019 16:08
This troubleshooting guide is the result of the outcome of issue rytilahti#422.

Please have a look at the issue for more insights:
rytilahti#422
@domibarton domibarton force-pushed the troubleshooting-guide branch from 5962903 to 4d33ef2 Compare January 20, 2019 16:09
@domibarton
Copy link
Contributor Author

This is the device: https://www.home-assistant.io/components/remote.xiaomi_miio/ the internal name in various pieces of code is chuangmi_ir.

@cnrd thanks, mentioned it in the docs!

Looks good, thanks! :-) You could add a link to the issue in case someone wants to read more about how you came to these conclusions.

@rytilahti added the link in the notes and pushed the branch again…

@rytilahti rytilahti merged commit 7eee203 into rytilahti:master Jan 20, 2019
@domibarton domibarton deleted the troubleshooting-guide branch January 24, 2019 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants