Steve Easley
Steve Easley
I wonder if this PR should be against dev and not your sensor PR. Its not clear what the change is to other reviewers. But maybe I am wrong.
When I commented here I didnt realize you had created PRs. So deleted the comment since I was addressing my questions in the PR. But thanks for the extra explanation....
@msavazzi Looks like you are pointing to the wrong documentation PR.
The documentation that was merged only talks about the binary sensory that was added a few weeks ago (https://github.com/home-assistant/home-assistant.io/pull/31058/files). I see nothing about the new sensors being added in this...
imo if we make it a Sensor Entity now, we can't easily add a Select Entity later without making the integration confusing for users (multiple places for the reported state),...
> Correct, but the point is not to add a Select Entity as is not how the Projector works :) Projector are: > > * Sensor (read only) > *...
Another way to think about this is that things like: - power state (warming, cooling, etc) - lamp time - color space - etc are all read only and make...
@msavazzi I do understand this would be a fair amount of work for you to implement if we do agree this is the correct route, especially since you don’t plan...
Thanks for the PR @anupmurarka. That fixes the two invalid remote codes, and I added a handful of new ones as well. Hopefully will get into the next HA release...
@home-assistant close