Rename import/export values to 'energy' instead of 'power'

DCSBL / ha-homewizard-energy

The old custom Home Assistant integration for HomeWizard Energy. This integration is now available in Core.

https://community.home-assistant.io/t/custom-component-homewizard-energy-wifi-p1-meter-and-kwh-meter/227441

Apache License 2.0

26 stars 3 forks source link

Rename import/export values to 'energy' instead of 'power' #55

Closed DCSBL closed 3 years ago

DCSBL commented 3 years ago

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Resolves #53
What is the current behavior? (You can also link to an open issue here) Import/export values are named with power
What is the new behavior (if this is a feature change)? Renamed default names for import/export values to energy Also updated capitalization.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) As the API name is not renamed, it shouldn't break. Home Assistant should take care of the renamed entity

github-actions[bot] commented 3 years ago

🎉 HACS repository validator action summary 🎉 ✅ The repository is not archived ✅ manifest.json file exist ✅ All required keys are present in manifest.json ✅ hacs.json has the 'name' key set ✅ The repository has a description ✅ homewizard_energy is added to https://github.com/home-assistant/brands NICE! ✅ The repository has topics ✅ The repository has issues enabled ✅ This day ends with an 'y' (Friday) ✅ README.md exists ✅ Requirements validation ✅ Python wheels ✅ HACS load check

DCSBL commented 3 years ago

Closed this one because lot of files were commited that shouldn't be there and force-push is a bad idea.