-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Config UI: add tariffs #26698
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: master
Are you sure you want to change the base?
Config UI: add tariffs #26698
Conversation
|
This will also simplify adding additional fees ;) |
cmd/setup.go
Outdated
| } | ||
|
|
||
| // Track which types are configured via YAML | ||
| yamlConfigured := make(map[api.TariffUsage]bool) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das ist viel Code fΓΌr yaml. KΓΆnnen wir das einfach nicht tun, fΓΌr teil-yaml Setups keine weitere UnterstΓΌtzung anbieten?
| }) | ||
| } | ||
|
|
||
| // Load device-based tariffs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wozu brauchts das? Die stehen in den tariffs.* Variablen doch schon drin?
server/http_config_tariff_handler.go
Outdated
| var res TariffRefs | ||
| if err := settings.Json(keys.TariffRefs, &res); err != nil { | ||
| // return defaults if not configured | ||
| res = TariffRefs{Currency: "EUR", Solar: []string{}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do this in line 24
server/http_config_tariff_handler.go
Outdated
| // return defaults if not configured | ||
| res = TariffRefs{Currency: "EUR", Solar: []string{}} | ||
| } | ||
| if res.Solar == nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove
server/http_config_tariff_handler.go
Outdated
|
|
||
| // Validate and update tariff references | ||
| if payload.Grid != nil { | ||
| if *payload.Grid != "" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this make sense? Either ignore in the first place or error. Loops instead of duplicated code?
|
Here the working ui for the new Bildschirmaufnahme.2026-01-19.um.20.18.53.webm |





Add form based configuration for tariffs and forecast to config ui.
π΅πβοΈπ°οΈ make grid, feedin, co2, solar and planner tariffs ui configurable
π add
fixedandfixed-zonestariff template. keeping separate to simplify standard setup use caseπ introduce
zonesparam type to specify hour/day/month based pricesβ‘ introduce
pricePerKWhparam to support currency specific inputct/kWh,rp/kWh, ...π forecast visualization, using existing price chart
π migration: no auto-migration, config via yaml (file or ui) still works, tariffs can be migrated one by one. only one source allowed per tariff type
TODOs
zonesparamScreenshots

config.ui.tariff.webm