search

nerrixde / DSBApi

API für die in Deutschland verbreitete DSBmobile App für Vertretungspläne an Bildungseinrichtungen
GNU General Public License v3.0
35 stars 6 forks source link

Reihenfolge der Felder geändert? #17

Closed cybcon closed 3 years ago

cybcon commented 3 years ago

Hi, aktuell bin ich am implementieren Eurer Bibliothek und mir ist dabei aufgefallen, dass bei uns in der Schule die Felder eine andere Reihenfolge haben.

Screenshot 2021-04-21 110957

Da ich das Ergebnis in einen Pandas DataFrame lutsche, habe ich als Workaround einfach die Spaltennamen getauscht. Wenn die Tabellen Überschriften einheitlich sind, könnte man evtl. ein dynamisches Feldnamen mapping einbauen. Leider fehlt mir hier die Vergleichsmöglichkeit.

Viele Grüße, Micha

nerrixde commented 3 years ago

Ja, jede Schule braut da leider ihren eigenen Brei.

cybcon commented 3 years ago

War ja klar :-D

wäre es eine Idee, statt dem entries.pop(0) in Zeile 87 einen Parser zu bauen, der ein Array aufbaut z.B. fields=list(), die erste Zeile parsed und "normalisiert" und dann den Feldnamen von dort holt? Man hätte dann auch gleich die Läge der Tabelle, könnte durch die fields loopen und das jeweilige value aus der gerade aktuellen Zeile ermitteln. Dann wären die Namen dann nicht mehr "eindeutig" und in "einfachem Englisch" aber dafür dynamisch und passend zur Schule.

cybcon commented 3 years ago

Hab nen Docker container daruas gebaut der das Ding per Pushover verschickt. https://hub.docker.com/r/oitc/dsb2pushover

Mal schauen ob ich das mit den dynmischen Tabellen ggf. konfigurativ hinbekomme.

nerrixde commented 3 years ago

👍

cybcon commented 3 years ago

Siehe neuen Pull request