search

home-assistant / architecture

Repo to discuss Home Assistant architecture
315 stars 99 forks source link

Adding platform to root integration #434

Closed dgomes closed 3 years ago

dgomes commented 4 years ago

Context

This issue comes https://github.com/home-assistant/core/pull/40028 where a PR was made to add the "Notify" platform to the TTS integration.

The proposed contribution is generic (it works with all existing and future tts platforms) and was therefore created under the tts integration.

Proposal

I'm proposing that root integrations can implement other existing platforms. This makes sense not only for the case that triggered this, but also for e.g. : cameras and image_processing

Consequences

No foreseen consequences, it simply has never been done before, but its based on existing required services, so there is guarantee of support by all inherited integrations.

balloob commented 3 years ago

It has been done before with Switch having a Light platform. Or Group having a light and cover platform. This is fine.