Closed elupus closed 1 week ago
Hey there @dgomes, mind taking a look at this pull request as it has been labeled with an integration (integration
) you are listed as a code owner for? Thanks!
I'm OK with adding an option to change the integration method, but:
It very easy to get wrong when you initially add it. Also replacing the source sensor is allowed for stuff like group helpers, so see no reason not to add it.
I have removed the scaling and time unit config, as well as making sure the new entity is selected has a compatible unit of measurement.
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks :+1:
Proposed change
Allow changing a riemann sum integral after it's been added from config flow.
Type of change
Additional information
Checklist
ruff format homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: