feat(finance): org fee for old participants
#149
Merged
christian.merten
merged 12 commits from MK/org_beitrag into main 8 months ago
Reviewers
Request review
No reviewers
Labels
Clear labels
Betrifft die Administration des Kompass
Betrifft Anwender:innen des Kompass, das heißt nicht-Admins
Betrifft die Entwicklungs- und Produktiv Infrastruktur
Bedarf keiner strukturellen Änderungen und ist leicht umsetzbar, weil es ähnliche Funktionen schon gibt
Breaking change that won't be backward compatible
Something is not working
Documentation changes
Improve existing functionality
New functionality
This is security issue
Issue or pull request related to testing
Betrifft die Kompass Produktivumgebung von Heidelberg
Betrifft die Kompass Produktivumgebung von Ludwigsburg
The priority is critical
The priority is high
The priority is low
The priority is medium
Issue has been confirmed
This issue or pull request already exists
Invalid issue
This issue won't be fixed
Somebody has started to work on this but abandoned work
The pull request or issue awaits a response by the author to questions or comments by reviewers.
Something is blocking this issue or pull request
Feedback is required to reproduce issue or to continue work
Apply labels
Effekt
Admins
Betrifft die Administration des Kompass
Effekt
Anwender
Betrifft Anwender:innen des Kompass, das heißt nicht-Admins
Effekt
Infrastruktur
Betrifft die Entwicklungs- und Produktiv Infrastruktur
Issue gut geeignet für Einsteiger:innen
Bedarf keiner strukturellen Änderungen und ist leicht umsetzbar, weil es ähnliche Funktionen schon gibt
Kind/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Lokal
Heidelberg
Betrifft die Kompass Produktivumgebung von Heidelberg
Lokal
Ludwigsburg
Betrifft die Kompass Produktivumgebung von Ludwigsburg
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Awaiting author
The pull request or issue awaits a response by the author to questions or comments by reviewers.
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No Label
Effekt
Admins
Effekt
Anwender
Effekt
Infrastruktur
Issue gut geeignet für Einsteiger:innen
Kind/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Lokal
Heidelberg
Lokal
Ludwigsburg
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Awaiting author
Status
Blocked
Status
Need More Info
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Set Project
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
No dependencies set.
Reference: digitales/kompass#149
Reference in New Issue
There is no content yet.
Delete Branch 'MK/org_beitrag'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
Allow deducting a configurable organisational fee for participants older than 27 from subsidies. This is calculated per day and old participant.
aktuell wird im Kompass noch nicht abgebildet, dass TN 27+ pro Person und Tag einen Org-Beitrag von 10€ bezahlen. Ich habe mich mal an einer Implementierung versucht.
Kurz: Anhand der Alters der TN bei der Ausfahrt wird der theoretische Org-Beitrag berechnet.
Sofern Aufwandsentschädigung oder Zuschuss in Anspruch genommen wird, wird der fällige Org-beitrag in der Kostenübersicht und bei der Abrechnung angezeigt und mit Zuschuss oder Aufwandsentschädigung verrechnet (ggf. als negative Überweisung erstellt).
Wenn weder Aufwandsentschädidung noch Zuschuss in Anspruch genommen wird, wird in diesem Ansatz auch kein Org-Beitrag erhoben:
Meinungen?
Sieht gut aus, danke! Ein paar Kommentare:
@ -342,3 +354,3 @@@propertydef total_subsidies(self):def total_org_fee_theoretical(self):Ist es so gedacht, dass halbe Tage mit 5€ berechnet werden?
das haben wir bisher so nicht gemacht, ich fände es aber fairer als explizit auf- oder abzurunden.
@ -114,2 +115,4 @@{% endif %}{% if total_org_fee %}{% blocktrans %}Since overaged people where part of the excursion, an organisational fee of 10,00€ per person per day has to be paid. This totals to {{ total_org_fee_theoretical }}€. This organisational fee will be accounted against allowances and subsidies.{% endblocktrans %}Was hältst du davon hier die Rechnung anzuzeigen? Also sowas wie "Daher werden insgesamt 10€ * N TN * T Tage = Gesamt € ..."? Oder ist es dann zu unübersichtlich?
ich habe den Rechenweg für den Org-Beitrag transparenter gemacht.
ja ich sehe auch die Notwendigkeit, die Übersichtlichkeit zu erhöhen. ggf. durch aufklappbare Bereiche für die unterschiedlichen Kostenbestandteile oder eine Checkliste am Ende, die etwaige Probleme oder Abhängigkeiten zusammenfasst.
Juleis, die als TN bei einer Ausfahrt dabei sind, bezahlen der Org-Beitrag
Danke!
Implementierung Org-Beitrag bei Ausfahrtento feat(finance): org fee for old participants 8 months ago@ -371,0 +384,4 @@"""participants older than 26.99 years need to pay a fee of 10€ per person per day."""if self.excursion is None:return 0return cvt_to_decimal(10 * self.excursion.duration * self.excursion.old_participant_count)Kannst du die hard gecodete
10in eine Konstante dersettingsverschieben?10kann gerne der default sein.hatte sogar schon eine Konstante org_fee angelegt, nur noch nicht im Code verwendet. Habe das jetzt nachgeholt. Dementsprechend bräuchte das Deployment dann auch die aktuelle production_config. Vielen dank fürs reviewen!
3250dc7089into main 8 months agoReviewers
3250dc7089.