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

Beschreibung für minsoc ändern? #17662

Open
VolkerK62 opened this issue Dec 8, 2024 · 6 comments
Open

Beschreibung für minsoc ändern? #17662

VolkerK62 opened this issue Dec 8, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@VolkerK62
Copy link
Contributor

Die Beschreibung in

- name: minsoc

bezieht sich auf die Fahrzeug Funktionalität, wird aber gleichzeitig auch bei dem steuerbaren Batteriespeichern genutzt (leider identische Bezeichnung). Dort passt die Beschreibung gar nicht.
Den Parameter minsoc gibt es zwar beim Vehicle noch, der wird doch aber nur noch im UI gesetzt.
Dann könnte man in der defaults.yaml die Beschreibung doch ändern, oder?

@naltatis naltatis self-assigned this Dec 8, 2024
@andig
Copy link
Member

andig commented Dec 10, 2024

Blöd... ich sehe 3 Möglichkeiten:

  1. Lowtech: Hilfstext bei jeder Verwendung neu und konsistent überschreiben. Nachteil: Pflegealptraum.
  2. EInfach: Eine der Parameterverwendungen umbenennen. Nachteil: muss in allen Templates gemacht werden, breaking Change für die Anwender (@naltatis oder fällt uns da eine Migration ein?)
  3. Hightech: Parameter identischen Namens aber unterschiedlicher Ausprägung Implementierung und je nach Kontext (einschließlich "fake" contextes wie z.B. "vehicle") anders interpretieren.

@andig andig added the enhancement New feature or request label Dec 10, 2024
@VolkerK62
Copy link
Contributor Author

zu 2) "vehicle minsoc" gibt es doch gar nicht mehr im Template, oder?

@naltatis
Copy link
Member

@VolkerK62 Ja das ist so. Das Minsoc Property am Fahrzeug ist deprecated und der hinterlegte Text kommt nirgends mehr zur Anzeige. Im UI kommt die entsprechende Beschreibung aus Weblate. Heißt wir können einfach den Beschreibungstext auf den neuen Anwendungsfall (Batteriespeicher) anpassen.

@andig
Copy link
Member

andig commented Dec 11, 2024

Variante 0: einfach das Problem lösen. Top!

@andig
Copy link
Member

andig commented Dec 11, 2024

@naltatis doch nicht (ganz) top. Das ist ein temporärer Zustand- perspektivisch würde das Setting ja wieder in die Gerätekonfig wandern.

@naltatis
Copy link
Member

@andig nicht schlimm. Selbst wenn wir den MinSoc später wieder ans Fahrzeug (Config UI) verschieben fänd ich es nicht schlimm das zu einem festen Feld in der UI zu machen. Das gilt ja für alle Fahrzeuge (außer Offline).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants