search

CoolKit-Technologies / ha-addon

Apache License 2.0
265 stars 121 forks source link

Set "Integration" for devices/entities to "eWeLink" f.e. #75

Closed Masterz69 closed 2 years ago

Masterz69 commented 2 years ago

Add-on related entities should have "Integration" set to add-on name.

This is actually current practice, used widely. Setting Integration=Sensor for sensor and so on is wrong idea.

With best wishes.

coolkit-tony commented 2 years ago

Hello, @Masterz69. Thanks for your feedback. Do you mean the entity_id should be changed?

niezw commented 2 years ago

Can you tell us in more detail to help us better understand what you mean? thank you for your reply! @Masterz69

Masterz69 commented 2 years ago

image

Will on example. You can see on screenshot devices & entities created by same Integration. All of them have property 'Integration' set to Integration name. Actually this common practice in HA. As far I see.

niezw commented 2 years ago

We agree with you.

But our AddOn doesn't create entity in the same way as integration. We use REST API.

integration name as "Switch", is the default of HA .We can't make any changes.

niezw commented 2 years ago

If you have any other questions, please create new issues. Thank you for your reply.