[mqtt-homeassistant] Add JOSNPath and JINJA as requirements for MQTT HA

[fundef1]

cfr. forum post: https://community.openhab.org/t/mqtt-homeassistant-thing-detected-no-channel/111775/

[J-N-K]

If it‘s a requirement, it should be installed automatically by adding a dependency to the feature.

[kaikreuzer]

@J-N-K No, we do not support dependencies between add-ons.
We might think about moving transformations to core, though and allow them to be required by add-ons, similar to the io-transport features - but that'll be a bigger change.

[J-N-K]

@kaikreuzer Why is that? I‘m pretty sure it would work.

[kaikreuzer]

Many reasons, in general known as dependency hell.

[mhilbush]

Ruh roh! This was done a long time ago when the transformations were part of ESH...

[J-N-K]

@mhilbush Thanks. I knew I saw that somewhere.

[Skinah]

I got caught out by the missing JINJA requirement, and in the last month I have seen at least 2 people in the forum asking why it does not work correctly with Tasmota devices. I am happy to change the feature.xml and do the testing but only if this is the recommended way forward?

https://community.openhab.org/t/mqtt-config-with-auto-discovery/109955

[openhab-bot]

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/oh3-mqtt-homeassistant-discovered-aquara-window-contact-not-working/113861/4

[fwolter]

If I understand correctly, the right way is to document it and not to add it to the features.xml.

[fwolter]

@fundef1 Can you sign-off your commit or post the sign-off message here if that is easier for you? See https://www.openhab.org/docs/developer/contributing.html#sign-your-work

[fundef1]

Signed-off-by: Jo Degraef jo.degraef@addestino.be