search

hitobito / hitobito_cevi

A hitobito wagon defining the organization hierarchy and additional features for Cevi
Other
6 stars 3 forks source link

BSV-Export: Problem mit Kantone #115

Open tschuepbach opened 1 year ago

tschuepbach commented 1 year ago

Aktuelles Problem

Beim BSV-Export werden nur die TN gezählt, bei welchen ein Kanton (auf dem Profil) definiert ist.

Test mit folgendem Kurs: Frösch-Up 2023 Anz. Teilnehmende: 28, davon sind alle im BSV-Alter (bis 30-jährig)

Export der TN-Liste (anonymisiert): TN-Liste Frösch-Up 2023.xlsx

Gemäss BSV-Export werden aber nur 14 Teilnehmende angegeben: bsv_export_test_frösch-up.csv

Der Kanton ist bei uns pro Person spezifisch, wird nicht automatisch berechnet und kann bei Events nicht als Pflichtangabe definiert werden.

Unser BSV-Export kommt direkt aus hitobito_youth. Die relevanten Stellen sind wohl

Da wir von der GS Cevi Schweiz nun definitiv mit dem Export arbeiten, möchten wir das Problem mit den Kantonen lösen. Die Kosten dafür werden direkt von der GS Cevi Schweiz übernommen.

Ich sehe folgende zwei Möglichkeiten: (Für bräuchten in jedem Fall zu gegebener Zeit eine grobe Zeit- resp. Kostenschätzung)

tschuepbach commented 1 year ago

Möglichkeit 1

Falls möglich mit einem "Schnellfix" sämtliche Teilnehmende (bis 30 jährig, wohnhaft in der Schweiz) beim BSV-Export berücksichtigen.

Bedingung: Müsste bis spätestens Ende Juli auf der Produktivumgebung sein.

Carbon hatte die Idee :canton_count hier zu entfernen: https://github.com/hitobito/hitobito_youth/blob/2d1736867cde60f49c41f50d2c8b5ddafce5484f/app/domain/export/tabular/events/bsv_row.rb#L12 Allenfalls löst dies das Problem schon..

Wichtig: Die Anpassungen dürfen auf keinen Fall Einfluss auf die BSV-Exporte der anderen Jugendverbände haben! Z.B. arbeitet die PBS noch mit dem erweiterten BSV-Export (wo die Kantone gebraucht werden), bei der Jubla wird der Kanton automatisch berechnet. Diskussion ist hier zu finden: https://github.com/cevi/hitobito_cevi/issues/180

tschuepbach commented 1 year ago

Möglichkeit 2

Fall eine schnelle Lösung bis Ende Juli nicht möglich ist, werden wir gemeinsam mit PBS & Jubla im Verlauf Q3/Q4 die sonst noch nötigen Anpassungen beim BSV-Export angehen. Siehe dazu: https://github.com/hitobito/hitobito/issues/1933

carlobeltrame commented 1 year ago

Zur Kontrolle ob ich euer Problem richtig verstanden habe, formuliere ich es hier nochmals in eigenen Worten: Im BSV-Export werden bei der TN-Anzahl aktuell nur Personen gezählt, welche einen Kanton eingetragen haben. Neu wollt ihr auch Personen zählen, welche keinen Kanton eingetragen haben (solange sie nicht älter als 30 sind und in der Schweiz wohnhaft sind). Stimmt das so?

Wenn ja, dann sind die relevanten Stellen im Code der Filter wo die nicht in der Schweiz wohnhaften TN von der Zählung ausgeschlossen werden sowie die Definition von "in der Schweiz wohnhaft". Aktuell zählen eben im Youth-Wagon nur diejenigen Personen als "in der Schweiz wohnhaft", welche einen Kanton eingetragen haben. Es wäre einfach, im Cevi-Wagon diese Unterscheidung durch andere Logik zu ersetzen (indem wir im Cevi-Wagon eine andere Implementation für ch_resident? einbauen), sicher einfacher als der Abspracheaufwand eurer Möglichkeit 2. Aber ihr müsstet dazu eine Definition liefern, wie hitobito beim Cevi Personen erkennen soll, die (nicht) in der Schweiz wohnhaft sind.

nchiapol commented 1 year ago

@carlobeltrame Ob das Problem exakt formuliert ist, muss @tschuepbach bestätigen.

Mein Vorschlag für "nicht in der Schweiz wohnhaft": Bei der Adresse ist das Select-Feld Land nicht leer und nicht Schweiz.

tschuepbach commented 1 year ago

Zur Kontrolle ob ich euer Problem richtig verstanden habe, formuliere ich es hier nochmals in eigenen Worten: Im BSV-Export werden bei der TN-Anzahl aktuell nur Personen gezählt, welche einen Kanton eingetragen haben. Neu wollt ihr auch Personen zählen, welche keinen Kanton eingetragen haben (solange sie nicht älter als 30 sind und in der Schweiz wohnhaft sind). Stimmt das so?

Absolut korrekt.

Den Vorschlag von Carbon finde ich gut. Wichtig wäre, dass auch Personen, bei denen das Feld leer ist, als "wohnhaft in der Schweiz" gezählt werden. Bei Stichproben von drei Kursen gibt es etliche, welche kein Land definiert haben.

Falls noch einfacher umzusetzen, würde ich sogar vorschlagen, temporär einfach ALLE Personen U30 mitzuzählen. Das Aussortieren von Personen, welche kein Wohnsitz in der Schweiz haben, ist für mich kein grosser Aufwand, resp. weiss ich, welche Kurse davon betroffen sind.

Anschliessend könnten wir im Verlauf der Umsetzung von Möglichkeit 2 uns in Ruhe überlegen, wie wir dies mit den Kantonen/Wohnsitz handhaben möchten.

tschuepbach commented 1 year ago

Ich denke hier ist nichts in der Pipeline oder?

Somit alle Arbeiten stoppen, die Frist ist abgelaufen. Wir werden im Rahmen der Möglichkeit 2 unsere Anliegen einbringen und umsetzen.