-
-
Notifications
You must be signed in to change notification settings - Fork 705
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
Add note about leading zero on Fiat pin #17788
base: master
Are you sure you want to change the base?
Conversation
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.
Lgtm
Das gilt für jeden anderen Wert in yaml auch, müssten wir dann also überall schreiben. Vielleicht wäre es sinnvoller- wenn die Anzahl der Ziffern bekannt ist- führende Nullen einfach zu ergänzen? Das kann auch im Template passieren und schließt die Fehlerquelle vollständig aus. |
Gibt es eine Möglichkeit den YAML Parser über ein Schema (oder ähnliches) dazu zu zwingen den Wert immer als String zu nehmen? |
Nein |
Ja, oder in dem Fall einfach ne Fehlermeldung zu spielen, dass die Eingabe zu kurz ist. Das würde das Fehlerfinden im vergleich zu heute deutlich verbessern. Das Problem mit den führenden Nullen wird im Config UI ohnehin kein Problem mehr sein. In "evcc configure" könnte man automatisch Anführungszeichen ergänzen, weiß nicht, wie sich das heute verhält. |
Man könnte dem Template eine ganz einfache // "{{ leadingzero .pin 4 }}"
func leadingzero(s string, l int) string {
return strings.Repeat("0", max(0, l-len(s))) + s
} |
Related to #12404
If the pin starts with a zero, it may not function properly when not set in quotes.