cevi / hitobito_cevi

repository is used to manage requirements and issues for db.cevi.ch
https://db.cevi.ch
Other
2 stars 0 forks source link

BSV-Export: Probleme mit Kantonen #180

Open tschuepbach opened 1 year ago

tschuepbach commented 1 year ago

Wir möchten auf der GS CCH ab diesem Jahr mit dem BSV-Export arbeiten, weil dies den Aufwand für sämtliche Beteiligte massiv verkleinert.

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

Aktuelles Problem

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

Die Kantone spielen eigentliche keine Rolle mehr, dies wird hier diskutiert: hitobito/hitobito/issues/1933

Gemäss Change-Log wurden die BSV-Exporte hier angepasst (auf Grund der Aufhebung vom Mindestalter):

Ist für jemanden von euch ersichtlich (im Code), warum nur Teilnehmende exportiert werden, bei denen ein Kanton hinterlegt ist? Oder war dies schon immer so und wir habe dies erst jetzt bemerkt?

Ein weiteres "Problem" ist, dass bei Kursen, der Kanton nicht als Pflichtfeld definiert werden kann:

image
tschuepbach commented 1 year ago

Heute per Zufall über folgendes gestolpert:

image
nchiapol commented 1 year ago

Kannst du beim ursprüngliche Anliegen direkt mit Safari schauen, wie sie das lösen?

Bei der fehlende Übersetzung müsste ich die betroffene Ansicht und Person wissen - könnte das ein in Import-Problem sein? (die korrekte Translation ist de.activerecord.attributes.cantons.ar)

tschuepbach commented 1 year ago

Ansicht: https://db.cevi.ch/groups/1/events/3217/participations/72146

Auf die Person habe ich keinen Zugriff. Via "Kontaktdaten ändern" auf der Ansicht oben, ist kein Kanton definiert.

Michael-Schaer commented 1 year ago

Ist für jemanden von euch ersichtlich (im Code), warum nur Teilnehmende exportiert werden, bei denen ein Kanton hinterlegt ist? Oder war dies schon immer so und wir habe dies erst jetzt bemerkt?

Ich könnte mir vorstellen, dass der Kanton pro Person Cevi spezifisch ist: https://github.com/hitobito/hitobito_cevi/blob/master/app/models/cevi/person.rb

Bei uns geht die Zuordnung normalerweise via Kantonalverband und von dort wiederum zum Kanton (Wir können unseren KVs unter "Gruppe Bearbeiten" Kantone zuordnen). Könnte das der Zusammenhang sein?

nchiapol commented 1 year ago

Ansicht: https://db.cevi.ch/groups/1/events/3217/participations/72146 Auf die Person habe ich keinen Zugriff. Via "Kontaktdaten ändern" auf der Ansicht oben, ist kein Kanton definiert.

Das war wohl tatsächlich ein Import-Problem. Ich konnte den Wert ändern, jetzt scheint alles korrekt.

Ich könnte mir vorstellen, dass der Kanton pro Person Cevi spezifisch ist
[...] Bei uns geht die Zuordnung normalerweise via Kantonalverband und von dort wiederum zum Kanton

Danke für die Gedanken. Unser BSV-Export kommt direkt aus hitobito_youth. Die relevanten Stellen sind wohl https://github.com/hitobito/hitobito_youth/blob/2d1736867cde60f49c41f50d2c8b5ddafce5484f/app/domain/export/tabular/events/bsv_row.rb#L12 und https://github.com/hitobito/hitobito_youth/blob/2d1736867cde60f49c41f50d2c8b5ddafce5484f/app/domain/bsv/info.rb#L48

Wenn die anderen Verbände einverstanden sind, liesse sich das also wohl rasch via Pull-Request lösen. (Ich nehme an, es reicht :canton_count, aus dem ersten File zu löschen.

(und dann den rest des Codes aufzuräumen ;-) Ist aber auch recht Überschaubar: https://github.com/search?q=repo%3Ahitobito%2Fhitobito_youth%20canton_count&type=code

tschuepbach commented 1 year ago

Wenn die anderen Verbände einverstanden sind, liesse sich das also wohl rasch via Pull-Request lösen. (Ich nehme an, es reicht :canton_count, aus dem ersten File zu löschen.

Das klingt doch vielversprechende. 😇 Vielen Dank. Klärst du dies mit den anderen Verbänden oder soll ich dies tun?

nchiapol commented 1 year ago

Ich komme aktuell nicht dazu mich darum zu kümmern. Ich denke, am effizientesten wäre es einen Pull-Request zu erstellen und dann die anderen Verbände auf diesen hinzuweisen.

Michael-Schaer commented 1 year ago

Ich müsste abklären, ob wir den Kanton bei uns noch brauchen. Wir haben noch den erweiterten BSV-Export. Dort brauchen wir den Kanton sicher drin. Ich weiss aber gerade nicht, ob der vom normalen BSV-Export erbt oder nicht.

tschuepbach commented 1 year ago

Danke für die Gedanken. Unser BSV-Export kommt direkt aus hitobito_youth. Die relevanten Stellen sind wohl https://github.com/hitobito/hitobito_youth/blob/2d1736867cde60f49c41f50d2c8b5ddafce5484f/app/domain/export/tabular/events/bsv_row.rb#L12 und https://github.com/hitobito/hitobito_youth/blob/2d1736867cde60f49c41f50d2c8b5ddafce5484f/app/domain/bsv/info.rb#L48

Wenn die anderen Verbände einverstanden sind, liesse sich das also wohl rasch via Pull-Request lösen. (Ich nehme an, es reicht :canton_count, aus dem ersten File zu löschen.

@richardjubla: Wie sieht dies bei euch aus?

richardjubla commented 1 year ago

Unsere Situation @tschuepbach :

Bei uns wird der Kanton in der Regel durch die Datenbank ausgefüllt. (Feld Kanton -> Wird aus der PLZ berechnet falls leer) Für Kurse gilt die Anweisung, die PLZ/Ort zwingend abzufragen -> Was zu einem ausgefüllten Feld "Wohnkanton" führt. Bei BSV-Export Kurse sollen wir demzufolge diesen Fehler nicht entdecken.

Probleme bereiten uns Profile, in denen das Feld Kanton mit dem ausgeschriebenen Namen anstelle des Kürzels ausgefüllt wird. Die Kursadministration kontrolliert dazu in der Spalte "Wohnkanton" in der Liste "Teilnehmende" eines Kurses die korrekten Angaben. image Hint: Hilfstext zum Feld Wohnkanton / Die Kursadmin "braucht" das Feld in der Anmeldeliste als zusätzliche Spaltenauswahl https://github.com/hitobito/hitobito/issues/2007

Konkret: Ich sehe im Pull-Request kein Problem, würde aber schon gerne sicherstellen, dass Hitobito die Abhängigkeiten/Qualitätskontrolle macht.