Skip to content

Support for the Xiaomi Mi WiFi Repeater 2 added#278

Merged
syssi merged 5 commits intorytilahti:masterfrom
syssi:feature/xiaomi-repeater-v2
Mar 26, 2018
Merged

Support for the Xiaomi Mi WiFi Repeater 2 added#278
syssi merged 5 commits intorytilahti:masterfrom
syssi:feature/xiaomi-repeater-v2

Conversation

@syssi
Copy link
Copy Markdown
Collaborator

@syssi syssi commented Mar 26, 2018

This implementation is the foundation for the xiaomi miio device tracker: https://github.com/syssi/xiaomi_repeater

(Closes: #275)

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.6%) to 68.707% when pulling 119f4de on syssi:feature/xiaomi-repeater-v2 into 1c46795 on rytilahti:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Mar 26, 2018

Coverage Status

Coverage increased (+0.4%) to 69.657% when pulling 5d9edff on syssi:feature/xiaomi-repeater-v2 into 1c46795 on rytilahti:master.

def test_configuration(self):
self.device._reset_state()

assert repr(self.configuration()) == repr(WifiRepeaterConfiguration(self.device.start_config))
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

line too long (102 > 100 characters)

Copy link
Copy Markdown
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.

Just one question, otherwise feel free to merge! 👍

self.data = data

@property
def access_policy(self) -> int:
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Is it known what this integer represents?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Unfortunately, no. My assumption is the connectivity between the clients. I cannot be controlled at the moment. The method is unknown / unused of the Mi Home app.

@syssi syssi merged commit e307486 into rytilahti:master Mar 26, 2018
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