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

[miio] suggest correct devicenames in readmehelper #9379

Merged
merged 6 commits into from Dec 27, 2020

Conversation

marcelrv
Copy link
Contributor

Signed-off-by: Marcel Verpaalen marcel@verpaalen.com

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
@marcelrv marcelrv added the enhancement An enhancement or new feature for an existing add-on label Dec 14, 2020
Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
Comment on lines +79 to +81
"aiicn.curtain.r51": "智能窗帘控制器",
"aiicn.light.pwm2": "LED驱动器",
"aiicn.light.r01": "单路接收器",
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure that chinese names would really help english speakers know what they are looking at...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm sure not... these devices don't have an English description in the cloud, guess they are only for sale in CN.
This list is all devices that are known for Xiaomi, that does not mean they are included in OH..

Dunno an easy fix for this (I can delete all non-english ones, but that does not solve anything)

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not really sure there is a good fix either. But I think as long as you only show the device name as additional information for the device and not as the key identifier for the device, both in openHAB and in documentation, then I think we should be ok.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When I actually implement such a device I normally use the English names. Until very recent I didn't even know what is the name of a device according to Xiaomi. I used a mix of commercial names ( how they call the devices on the webshops) or generic names based on the model e.g aiicn.light.r01 would become aiicn light, and it still unclear Google translate as final solution

Long story short... When I actually add a device, I'll add it with an English name.
The inclusion here in the readme maker is just to speedup/simplify the creation of the actual Java devicelist file.

@cpmeister cpmeister merged commit 0874df7 into openhab:main Dec 27, 2020
@cpmeister cpmeister added this to the 3.1 milestone Dec 27, 2020
seaside1 pushed a commit to seaside1/openhab-addons that referenced this pull request Dec 28, 2020
* [miio] suggest correct devicenames in readmehelper
* add newline at file end to make git happy
* [miio] update logic for mapping in readmeComments
* Allow readme maker to fix mappings based on the current options.
* [miio] make readmeOptionMapping static

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
seaside1 pushed a commit to seaside1/openhab-addons that referenced this pull request Dec 28, 2020
* [miio] suggest correct devicenames in readmehelper
* add newline at file end to make git happy
* [miio] update logic for mapping in readmeComments
* Allow readme maker to fix mappings based on the current options.
* [miio] make readmeOptionMapping static

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
Signed-off-by: Joseph Hagberg <joseph@zoidberg.se>
nowaterman pushed a commit to nowaterman/openhab-addons that referenced this pull request Jan 19, 2021
* [miio] suggest correct devicenames in readmehelper
* add newline at file end to make git happy
* [miio] update logic for mapping in readmeComments
* Allow readme maker to fix mappings based on the current options.
* [miio] make readmeOptionMapping static
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
* [miio] suggest correct devicenames in readmehelper
* add newline at file end to make git happy
* [miio] update logic for mapping in readmeComments
* Allow readme maker to fix mappings based on the current options.
* [miio] make readmeOptionMapping static

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
* [miio] suggest correct devicenames in readmehelper
* add newline at file end to make git happy
* [miio] update logic for mapping in readmeComments
* Allow readme maker to fix mappings based on the current options.
* [miio] make readmeOptionMapping static

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants