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

Add note about leading zero on Fiat pin #17788

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

lociii
Copy link

@lociii lociii commented Dec 18, 2024

Related to #12404

If the pin starts with a zero, it may not function properly when not set in quotes.

Copy link
Collaborator

@GrimmiMeloni GrimmiMeloni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@andig
Copy link
Member

andig commented Dec 18, 2024

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.

@andig andig marked this pull request as draft December 18, 2024 17:42
@GrimmiMeloni
Copy link
Collaborator

GrimmiMeloni commented Dec 18, 2024

Gibt es eine Möglichkeit den YAML Parser über ein Schema (oder ähnliches) dazu zu zwingen den Wert immer als String zu nehmen?

@andig
Copy link
Member

andig commented Dec 18, 2024

Nein

@naltatis
Copy link
Member

naltatis commented Dec 19, 2024

wenn die Anzahl der Ziffern bekannt ist- führende Nullen einfach zu ergänzen?

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.

@andig
Copy link
Member

andig commented Dec 19, 2024

Man könnte dem Template eine ganz einfache leadingzero Funktion verpassen:

// "{{ leadingzero .pin 4 }}"
func leadingzero(s string, l int) string {
	return strings.Repeat("0", max(0, l-len(s))) + s
}

@github-actions github-actions bot added the stale Outdated and ready to close label Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Outdated and ready to close
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants