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

Statistik und Bestandesmeldung um das Total pro Geschlecht ergänzt #54

Merged
merged 3 commits into from Apr 11, 2017

Conversation

Projects
None yet
4 participants
@jonasrutishauser
Contributor

jonasrutishauser commented Mar 18, 2017

Abteilung

Vorher: image
Nachher:
image

Kanton/PBS

Vorher: image
Nachher:
image

@RolandStuder RolandStuder requested a review from kronn Mar 20, 2017

@kronn

kronn approved these changes Mar 22, 2017

Mir ist nicht ganz klar, was mit Mitgliedern passiert, die kein Geschlecht angegeben haben. Ich vermute, dass das eher ein theoretisches Problem ist, was auch wenig mit der aktuellen Änderung zu tun hat. Wenn man diesen Fall auch abbilden möchte, müsste man natürlich noch eine entsprechende Zeile einfügen. So, wie der PR ist, ist es aber definitiv eine Verbesserung.

@RolandStuder

This comment has been minimized.

Show comment
Hide comment
@RolandStuder

RolandStuder Mar 22, 2017

Member

@kronn Geschlecht ist eine Pflichtangabe bei der Bestandesmeldung. Und auf dieser basieren diese Zahlen. Ist daher kein Problem

Member

RolandStuder commented Mar 22, 2017

@kronn Geschlecht ist eine Pflichtangabe bei der Bestandesmeldung. Und auf dieser basieren diese Zahlen. Ist daher kein Problem

@kronn kronn merged commit 7368394 into hitobito:master Apr 11, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jonasrutishauser jonasrutishauser deleted the PBSMiData:feature/statistic-total branch Apr 11, 2017

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