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

BSV-Export: Problem mit Kantone #115

Open
Tracked by #131
tschuepbach opened this issue Jun 29, 2023 · 6 comments
Open
Tracked by #131

BSV-Export: Problem mit Kantone #115

tschuepbach opened this issue Jun 29, 2023 · 6 comments

Comments

@tschuepbach
Copy link

tschuepbach commented Jun 29, 2023

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
Copy link
Author

tschuepbach commented Jun 29, 2023

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: cevi#180

@tschuepbach
Copy link
Author

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: hitobito/hitobito#1933

@carlobeltrame
Copy link
Member

carlobeltrame commented Jul 3, 2023

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
Copy link
Collaborator

nchiapol commented Jul 3, 2023

@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
Copy link
Author

tschuepbach commented Jul 3, 2023

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
Copy link
Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants