search

crowbarz / ha-sql_json

Updated SQL integration for Home Assistant that supports JSON attributes
7 stars 2 forks source link

Ability to configure via UI and/or reload sql_json yaml changes #10

Open bjornbak opened 2 months ago

bjornbak commented 2 months ago

It is a slow process to setup and test sql_json sensor when I have to restart Home Assistant every time.

Would it be possible to configure in the integration UI like the build in sql integration?

And/or add sql_json reload to the "YAML configuration reloading" list in Developer tools?

Other HACS integrations like pyscript has a reloader option there.

crowbarz commented 1 month ago

This integration was originally created as a patch on top of the built-in sql integration, but has not been rebased in ~years. I see that in the meantime the upstream SQL integration has undergone major changes including conversion to async and adding UI config support.

I will see if the patch can be applied to the current upstream version relatively easily, but given the major upstream changes this is unlikely - and thus will probably not happen any time soon.

In the meantime, I would suggest developing the appropriate SQL query using an external tool like DBeaver, rather than using this integration.

bjornbak commented 1 month ago

Another nice option is ofc to upstream the json feature :D

crowbarz commented 1 month ago

If you have the (not insignificant) cycles to complete the due diligence required to upstream this feature, then please feel free to contribute.