hass_nibe icon indicating copy to clipboard operation
hass_nibe copied to clipboard

Integration into HA core?

Open MatthiasLohr opened this issue 2 years ago • 7 comments

Hi,

I tried to check for similar tickets but couldn't find one, therefore here my question: Is there any specific reason why you don't want to add this integration to HA core?

I'm a HA contributor myself, highly interested in getting this integration into HA core, so if you agree, we could work together on getting your work into HA core. What do you think about that?

Best regards Matthias

MatthiasLohr avatar Apr 25 '22 11:04 MatthiasLohr

Just not been too find of the review experience :). Also i don't like some of my own design solutions for how it initializes (delays ha startup). That needs to be resolved before it can be mainlined.

Secondly it will need to ve mainlined in pieces since a big bang won't be accepted. That said, it would be nice to get it in.

elupus avatar Apr 25 '22 13:04 elupus

So, then, with your permission, I would take your code and prepare something towards a first pull request to HA core. I already have one successful integration built up from scratch and got accepted to HA core, so why not do another one :) If you're interested, we can also do some call and you could explain what you mean with design decisions.

MatthiasLohr avatar Apr 25 '22 13:04 MatthiasLohr

Go for it!

elupus avatar Apr 25 '22 13:04 elupus

Ps. I might even end up reviewing my own code ;)

elupus avatar Apr 25 '22 13:04 elupus

Meeh. Just discovered: My new NIBE devices are using the new myUplink API. This raises the question: Should we try to combine it all in one python library (by extending your nibeuplink package) and HA integration, or should that be done in seperate integrations/libraries?

MatthiasLohr avatar Apr 26 '22 08:04 MatthiasLohr

Hey,

Has there been any progress in getting this integration in Core? Any timeline?

Thanks for all the hard work! /C

Chrille144 avatar Sep 15 '22 06:09 Chrille144

Nope.

elupus avatar Sep 15 '22 06:09 elupus