Persist language preference #11446
Persist language preference #11446
Conversation
| @@ -95,6 +95,9 @@ type Site struct { | |||
| batterySoc float64 // Battery soc | |||
| batteryMode api.BatteryMode // Battery mode | |||
|
|
|||
| // user settings | |||
| language string // Language | |||
There was a problem hiding this comment.
Bin ich jetzt kein so großer Fan von. Muss das im Backend sein? Und falls ja: dann sollte es sich vmtl. auch auf den CSV Export auswirken?
There was a problem hiding this comment.
Wobei das eher locale denn Sprache wäre…
There was a problem hiding this comment.
Und falls ja: dann sollte es sich vmtl. auch auf den CSV Export auswirken?
Ja, aktuell wird beim CSV Aufruf die Sprache explizit übergeben. Das könnte man durch die eingestellte Sprache ersetzen. Allerdings bräuchte der Endpunkt dann noch ne Logik, um den "automatisch" (default) Fall zu handeln. Momentan bekommt der immer einer valide Sprache übergeben.
Daher habe ich das in diesem Schritt noch nicht angefasst und hab erstmal die existierende Logik für CSV gelassen.
There was a problem hiding this comment.
Locale vs. Language: Hier gehts wirklich nur um Sprache, so wie wir sie in Weblate abbilden.
| } | ||
|
|
||
| // SetLanguage sets the language | ||
| func (site *Site) SetLanguage(lang string) error { |
There was a problem hiding this comment.
Sollte es eine Rolle spielen, ob wir die Sprache überhaupt haben?
There was a problem hiding this comment.
Ja, wir nehmen hier Freitext an. Den sollten wir vorm Speichern validieren.
|
Mir erschließt sich generell der Bedarf nicht. Tradeoff code <> Nutzen. Meine bevorzugte Sprache hat schon das Betriebssystem und damit auch das evcc UI. |
|
Siehe verlinkter Issue. |
|
Ah, der Tesla Case. Warum ein deutsches Auto auf Holländisch eingestellt ist? Wohl eher ein Fall für den Tesla Support. |
Ja, wir können beim Tesla Case auch Wontfix sagen. Ursprünglich haben wir gesagt, wir machen das. Eine einfachere Lösung als das aktuelle sehe ich dafür momentan nicht. |
|
Closing this, since it introduces quite a lot of extra code in introduces new, maybe unwanted constraints. See #10459 (comment) This also means that we will won't fix #10459. Sorry. \cc @samsmooth @RTTTC |
fixes #10459
bhlang. addedbg🇧🇪 andtr🇹🇷Video: Firefox (en) <> Safari (de)
lang.persistance.mov