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: Probleme mit Kantonen #180

Open
tschuepbach opened this issue May 25, 2023 · 10 comments
Open

BSV-Export: Probleme mit Kantonen #180

tschuepbach opened this issue May 25, 2023 · 10 comments
Labels
HIGH Should be handled with highest priority needs-more-info

Comments

@tschuepbach
Copy link
Collaborator

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 tschuepbach added HIGH Should be handled with highest priority discuss Needs to be discussed in the cevi-db-gremium needs-more-info labels May 25, 2023
@tschuepbach
Copy link
Collaborator Author

Heute per Zufall über folgendes gestolpert:
image

@nchiapol
Copy link
Collaborator

nchiapol commented Jun 2, 2023

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

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

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

nchiapol commented Jun 6, 2023

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

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

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

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

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

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 hitobito/hitobito#2007

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

@nchiapol nchiapol removed the discuss Needs to be discussed in the cevi-db-gremium label Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HIGH Should be handled with highest priority needs-more-info
Projects
None yet
Development

No branches or pull requests

4 participants