PyLoxone icon indicating copy to clipboard operation
PyLoxone copied to clipboard

Code cleanup?

Open gigatexel opened this issue 10 months ago • 4 comments

Hi @JoDehli

If you would like, I could help you out on some code-cleanup like in this PR.

I found that not all entities had a device, and that not all parameters were consistent for each device. That's why I moved device-creation to __init.py__ so there is more uniformity and easier maintenance.

If you like this kind of work, I can also look into for example room/cat assignings. These also have various implementations.

gigatexel avatar Apr 02 '24 13:04 gigatexel

Not all entries should have a device. If I have time I look into the changes. But as I mentioned I am working on a complete new implementation. Until that I have to manually merge changes. This is a lot of work and maybe we can wait until this is finished.

JoDehli avatar Apr 02 '24 16:04 JoDehli

If you make a pull request you must first pull the latest version.ni saw that it is based on a older version of the master branch.

JoDehli avatar Apr 02 '24 17:04 JoDehli

If you make a pull request you must first pull the latest version.ni saw that it is based on a older version of the master branch.

Are you sure? I synced before branching. Github says its not behind.

image

gigatexel avatar Apr 03 '24 06:04 gigatexel

Not all entries should have a device. If I have time I look into the changes. But as I mentioned I am working on a complete new implementation. Until that I have to manually merge changes. This is a lot of work and maybe we can wait until this is finished.

No big deal to wait 👍

gigatexel avatar Apr 03 '24 06:04 gigatexel