Skip to content

Dynamic status page - #119

Merged
TheRealMoeder merged 7 commits into
mainfrom
dynamic_status
Apr 15, 2026
Merged

Dynamic status page#119
TheRealMoeder merged 7 commits into
mainfrom
dynamic_status

Conversation

@TheRealMoeder

@TheRealMoeder TheRealMoeder commented Apr 13, 2026

Copy link
Copy Markdown
Owner

Cherry-picking new status page from @selbstmacher #82

Also fixes a scaling bug for energy data in SMA parser.

@TheRealMoeder TheRealMoeder added the enhancement New feature or request label Apr 13, 2026
@TheRealMoeder

Copy link
Copy Markdown
Owner Author

@oldboys92 please test before I merge

@oldboys92

Copy link
Copy Markdown
Contributor

@TheRealMoeder tested successfully. LGTM. Here my summary:

  • I personally prefer having the unit after after the label instead of having it after the value (like this Voltage (V): 230.0), but it's really not a big issue
  • I think the unit for Energy Data (power) is easier to read in kWh (ex 2435.3 kWh instead of 2435343.2 Wh)🔍
  • The main page feels a little bit unfriendly to read on my big screen, I need to scroll down to read all information (even in 100%, I usually have 120%). Instead of additional rows for totals an additional column might solve it. On a phone, one needs to scroll anyway, since the columns are rendered differently.
  • Everything else works here, including the blinking of the blue led💡

@TheRealMoeder

Copy link
Copy Markdown
Owner Author

Thanks for your feedback. I modified the status to have 4 columns with dynamic scaling, changed units to kWh and also fix a long-time bug I never noticed.

@TheRealMoeder
TheRealMoeder merged commit d08baf6 into main Apr 15, 2026
4 checks passed
@TheRealMoeder
TheRealMoeder deleted the dynamic_status branch April 15, 2026 16:23
@TheRealMoeder TheRealMoeder mentioned this pull request Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants