Skip to content

Dev v.1.2.1#315

Merged
TypQxQ merged 5 commits intomainfrom
Dev-v.1.2.1
Feb 28, 2026
Merged

Dev v.1.2.1#315
TypQxQ merged 5 commits intomainfrom
Dev-v.1.2.1

Conversation

@TypQxQ
Copy link
Owner

@TypQxQ TypQxQ commented Feb 28, 2026

No description provided.

Q-Claw and others added 5 commits February 25, 2026 19:27
* Remove hardcoded icon for battery SOC sensor

Allow Home Assistant to apply dynamic battery icon based on state of charge.

* Remove hardcoded battery icons for other SOC sensors

Keep SOC sensors consistent and allow Home Assistant dynamic battery icons.

---------

Co-authored-by: Andrei Ignat <andrei@example.com>
Remove duplicate plant_total_charged_energy_of_the_evac
@greptile-apps
Copy link

greptile-apps bot commented Feb 28, 2026

Greptile Summary

This PR introduces v1.2.1 with UI/UX improvements and a bug fix for the Sigenergy integration.

  • Updated minimum Home Assistant version requirement to 2026.1
  • Standardized all number entities to use box input mode (NumberMode.BOX) instead of sliders for more precise value entry
  • Removed hardcoded battery icons from battery SOC sensors to allow Home Assistant's dynamic battery icons to display based on charge level
  • Fixed duplicate sensor definition for plant_total_charged_energy_of_the_evac

Confidence Score: 5/5

  • This PR is safe to merge with no identified risks
  • All changes are low-risk improvements: documentation update, UI preference standardization, icon optimization, and removal of a duplicate definition. No logical errors or breaking changes detected.
  • No files require special attention

Important Files Changed

Filename Overview
README.md Updated minimum Home Assistant version requirement from 2025.2 to 2026.1
custom_components/sigen/number.py Added NumberMode.BOX to all number entities for consistent UI input method across all controls
custom_components/sigen/static_sensor.py Removed hardcoded battery icons to enable dynamic icons, and fixed duplicate sensor definition

Last reviewed commit: 44060fe

@TypQxQ TypQxQ merged commit a3c3218 into main Feb 28, 2026
5 checks passed
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