OpenHAB Binding for AirGradient

junelac · August 16, 2024, 7:39am

Thanks ! Don’t worry, I’m mainly trying things on my new raspberry pi 2W. I will try to reinstall everything to see if it helps

Jorgen_Austvik · August 25, 2024, 4:56pm

Hi @junelac . I’ve been able to reproduce on 4.2.1 (doesn’t happen for me on 4.2.0).

I have a fix out here:

github.com/openhab/openhab-addons

[AirGradient] Convert from AbstractDiscoveryService to AbstractThingHandlerDiscoveryService

openhab:main ← austvik:airgradient_autodiscovery_fix

opened 04:54PM - 25 Aug 24 UTC

austvik

+12 -32

[AirGradient] Convert from AbstractDiscoveryService to AbstractThingHandlerDisco…veryService Because we get NPE on apiHandler in 4.2.1 (not on 4.2.0). This has been tested to not get the same NPE in 4.2.1, and is aligning the code to the new way to write discovery services. The reason AbstractDiscoveryService was used was to keep backwards compatibility when developing the binding outside the code base and it was installed from the addon store.

junelac · August 28, 2024, 11:08am

Thanks ! I will try it and give you my feedback as soon as I can

junelac · August 30, 2024, 9:53am

just to be sure: the changes are not yet available in the official add-ons (marked with a blue chck mark) ?

Jorgen_Austvik · August 31, 2024, 9:55am

I’m not sure if the blue checkmark takes the latest from the source code, or if it needs a new release of openhab to take effect.

I have made a jar available, which can be copied into the addons-folder to take effect if you want to try it out:

junelac · September 1, 2024, 9:57am

thanks, the addons works now !

Jorgen_Austvik · September 1, 2024, 7:17pm

Great! Thanks for testing it out! (Gives me peace of mind:)