evcc icon indicating copy to clipboard operation
evcc copied to clipboard

Sessions: fix loadpoint selection

Open Maschga opened this issue 6 months ago • 5 comments

Fix regression from #21047

✔️ Use correct type for loadpoints

\cc @andig @naltatis

Maschga avatar Jun 29 '25 07:06 Maschga

@Maschga can you add an e2e test that prevents this from happening again?

naltatis avatar Jun 29 '25 08:06 naltatis

Ich bekomme es nicht hin das <CustomSelect/> zu öffnen. :-/

Maschga avatar Jun 29 '25 09:06 Maschga

Hier sind auch die Label für Charging Point und Vehicle gar nicht korrekt gesetzt. Sind aktuell direkt ths. Ziel sollte sein, dass du sowas schreiben kannst:

modal.getByLabel("Charging point").selectOption("Carport");

naltatis avatar Jun 29 '25 14:06 naltatis

Wenn ich das in eine <label><input/></label> Struktur wie in der Dokumentation bringe, weiß ich nicht, wie ich das mit <th> und <tr> auflösen soll.

Maschga avatar Jun 29 '25 15:06 Maschga

Ein Input Element kann mehrere Label haben. Wir brauchen da ein th > label, dessen for auf die ID des inputs (select) zeigt. Sag Bescheid, wenn ich das machen soll weil das hier zu viel Ping-Pong ist :)

naltatis avatar Jun 29 '25 22:06 naltatis

Mach du es gerne. Wenn ich es sehe, werde ich es verstehen. 👍

Maschga avatar Jun 30 '25 07:06 Maschga