Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

base: add uptime sensor #281

Closed
wants to merge 2 commits into from
Closed

base: add uptime sensor #281

wants to merge 2 commits into from

Conversation

visigoth
Copy link

An uptime sensor is a nice-to-have debugging facility as it lets me verify that ratgdo is getting consistent power when using a voltage regulator connected to the battery backup terminals of my garage door opener.

@visigoth
Copy link
Author

I tested this with esphome run v25iboard.yaml and saw the uptime sensor in the table on the device's web page.

@bdraco
Copy link
Member

bdraco commented May 27, 2024

Uptime sensors are generally not enabled by default and usually only for debugging as they create many state updates and fill up Home Assistant's database quickly.

@visigoth
Copy link
Author

didn't know that; is there a way to have HA ignore it?

@ccutrer
Copy link

ccutrer commented Jun 28, 2024

You could set disabled_by_default: true in the ESPHome YAML, then a user would have to explicitly enable it in Home Assistant.

@bdraco
Copy link
Member

bdraco commented Jul 3, 2024

@visigoth Thanks for your PR. Because of the concern above #281 (comment) , I think we should not add this to the default config. Instead it would be nice to have a documentation PR on how to add it manually.

Thanks

@bdraco bdraco closed this Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants