Skip to content
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

Config UI: reset lp smart cost limit #19892

Merged
merged 1 commit into from
Mar 19, 2025
Merged

Config UI: reset lp smart cost limit #19892

merged 1 commit into from
Mar 19, 2025

Conversation

naltatis
Copy link
Member

fixes #19884

In settings nil float64 ist stored as "". This does not work with dynamic config json decode.

@naltatis naltatis added bug Something isn't working experimental Experimental feature labels Mar 17, 2025
@naltatis naltatis requested a review from andig March 17, 2025 23:03
@andig
Copy link
Member

andig commented Mar 18, 2025

Test failed und mir ist leider nicht klar was der Code anders macht- es wird immer noch als leerer String gespeichert?

@naltatis
Copy link
Member Author

es wird immer noch als leerer String gespeichert?

Im Fall evcc.yaml-LP wird's weiterhin als leerer String gespeichert.
Im Fall UI Config wird das jetzt als JSON null gespeichert. Vorher war das ein auch dort ein leerer String und das führt zur falschen Verhalten beim Einlesen. Der smartCostLimit *float64 in DynamicConfig wurde dann 0.0 und nicht nil.

@andig andig merged commit e87f4ac into master Mar 19, 2025
11 of 12 checks passed
@andig andig deleted the fix/smart_cost_config branch March 19, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working experimental Experimental feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Config UI: unable remove loadpoint price limit
2 participants