-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Refactoring] Update all Lumi (Aqara and Xiaomi) terminology, device vendor
and description
#6969
Conversation
Could you move them to |
I was thinking, maybe rename xiaomi.ts to lumi.ts, would be logical based on the manufacturer. |
+1 to renaming it to lumi.ts, this has been bothering me a bit lately while working on the TVOC sensor 😅 |
Agree! |
vendor
and description
vendor
and description
I would also suggest renaming to
|
+1 for renaming aqaraOpple to manuSpecificLumi to be inline with other manufacturer specific clusters |
Sounds like a good idea, I'll rename it to |
By analogy with zigbee-herdsman-converters/src/lib/xiaomi.ts Line 1588 in e4b88b8
|
Thanks! Totally missed that one. Renamed from I should also probably update OTA repo as well. |
The Aqara devices can be moved to a separate |
note to self; after this merge Koenkk/zigbee-herdsman#892 |
Yes, Lumi United is now fully independent from Xiaomi and makes all the Zigbee devices. Nowadays Xiaomi isn't even interested in Zigbee anymore it seems. |
|
You need to fix it locally by merging the master branch into your branch. |
Merged this one manually |
I think that you can also replace There is also another question: will we replace everything with prefix |
@Otnow What do you mean by that?
I'll check them, some of them were definitely Aqara Opple specific. |
…on; manufacturer code imports
@Otnow found and replaced manufacturerCode: 0x115f with imports. |
I meant something a little different, on the contrary, to shorten (remove
Of course, I would replace all |
@Otnow I see, but explicit variables look better to me. Either way it is better than using a number everywhere.
I'm currently looking into even more consolidation of functions with device model in names. Such as |
vendor
and description
vendor
and description
This is nice but I think it's better to do this in a separate PR, then we can merge this one quickly. |
@Koenkk I'm already doing it, but it is a monumental task, so I agree. I'll make a second PR after this and continue working on function refactoring. This one can be merged in a current state. |
vendor
and description
vendor
and description
Got a merge conflict again, can you get a look? Then I will merge this before doing any other work in |
I'm on it already. |
@Koenkk Probably fine now. |
Thanks! |
Idea
Cleaned up and standartaized all Aqara and Xiaomi devices
vendor
anddescription
. Added alternative model numbers aswhiteLabel
. Removed "Doesn't work as a router and doesn't support power meter" from descriptions, will move into Notes section.Term
xiaomi
was replaced with a more appropriatelumi
everywhere.New device distribution shown bellow.
Aqara
devicesSwitches
H1
D1
E1
Z1
US
Buttons
Sensors
Relays
Plugs
Lights
Curtains
Locks
Other
Xiaomi
devicesSwitches
Buttons
Sensors
Plugs
Locks