Add zigbee2mqtt

[o0shojo0o]

No description provided.

[GermanBluefox]

Automated adapter checker

ioBroker.zigbee2mqtt

👍 No errors found

• 👀 [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, zh-cn)
• 👀 [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, zh-cn)
• 👀 [W400] Cannot find "zigbee2mqtt" in latest repository

Add comment "RE-CHECK!" to start check anew

[Apollon77]

Hi,
hhere a short review feedback:

• not critical but if you use Async you shoulkd also await it ... or use the "Non async" method :-) e.g. https://github.com/o0shojo0o/ioBroker.zigbee2mqtt/blob/main/main.js#L55
• please also close the websocket/mqtt connections on unload

Rest I only fly over and looked ok ... Please check and adjust the above

[o0shojo0o]

Thank you for your review, I have fixed all the points.

[o0shojo0o]

@Apollon77 are there still open points?

[Apollon77]

Yes me finding time for recheck.

[o0shojo0o]

Add to stable @2.5.0

[o0shojo0o]

Update stable to v2.6.0

[Apollon77]

Sorry that the re-review too that long ... elcome in the repo!

One "note/tip" for unload: Alw<ys do the stuff first that should not throw any exception (like ending timers and intervals) and do other afterwards (or in separate try/catches) because as soon as one gives exception the rest of the code in the try/catch block is not executed. But thats optimization

[Apollon77]

oooooohhhhh @o0shojo0o one thing ... type "lighting"??? I think it is more or? type "hardware" like ziugbee adapter?

A change here would be "io.package change on github" and the PR here ... Can you please do that before I merge?
A new release is not urgently needed ... enough for next update

[o0shojo0o]

fixed ;)