Ergänzung Abrechnungs-Workflow
#150
Merged
christian.merten
merged 19 commits from MK/finance_workflow 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#150
Reference in New Issue
There is no content yet.
Delete Branch 'MK/finance_workflow'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
closes #92
Am Ende des Finanzworkflows gibt es aktuell noch einen Medienbruch:
Damit wir für unsere ausbezahlten Zuschüsse und Aufwandsentschädigungen bei der GS einen Beleg für die Buchhaltung einreichen können, soll dieser automatisiert im Kompass erzeugt werden können.
Ansatz: Sobald eine Abrechnung den Status bezahlt hat, kann in ihrer Detailansicht eine PDF heruntergeladen werden, die analog zum bisherigen Excel-Abrechnungsformular eingereicht werden kann. Hier wird detailliert die Berechnung der Zuschüsse, Aufwandsentschädigung und übernommene Ausgaben dargestellt und alle hinterlegten Belege als zusätzliche Seiten angefügt. Mit etwas Glück reicht das der GS zukünftig als einziger Beleg für übernommene Ausgaben.
weiterhin:
TODO:
Danke, sieht cool aus! Ich würde die Tabellen immer auf die volle Breite skalieren (zur Zeit sind sie zentriert und nur so breit wie nötig). Was denkst du?
@ -271,6 +271,11 @@ class Statement(CommonModel):def total_bills(self):Kannst du hier deine neue Funktion
bills_coveredverwenden?@ -0,0 +154,4 @@{% if statement.bills_covered %}{% for bill in statement.bills_covered %}{{ forloop.counter }}. {{ bill.short_description}} & {{ bill.explanation}} & {{ bill.paid_by.name }} & {{ bill.amount }} € \\Die Variablen müssen alle escaped werden (mit
{{ varname|esc_all }}) (auch noch weiter unten).@ -10,1 +10,3 @@<li>{% url opts|admin_urlname:'summary' original.pk|admin_urlquote as invite_url %}<a class="historylink" href="{% add_preserved_filters invite_url %}">{% trans 'Download summary' %}</a>Da fehlt denke ich ein
target='_blank'@ -119,1 +121,4 @@readonly_fields = ['text_length_warning']extra = 0def text_length_warning(self, obj):Diese Warnung wird zur Zeit nur hier angezeigt oder? Soll die auch in der Bestätigungsübersicht angezeigt werden? Oder sogar das Bestätigen nicht möglich sein, falls ein Text zu lang ist?
Die Idee war, dass es eher als Hinweis dient. Bei Zusammenfassung sind die Verwendungszwecke oft länger, so können sie eigenverantwortlich eingekürzt werden. Wenn sie trotzdem zu lang sind, passiert auch nichts, im zweifel schneidet die Banking-App den Text ab.
finde ich auch ansprechender, scheitert bei mir eher an der Komplexität der Umsetzung 🫢
ich schau es mir nochmal an
Habe ich gefixed und hoffentlich das ganze setup vereinfacht.
Danke! Ich bin zufrieden, da ich jetzt aber ein wenig in deinem PDF rumgepfuscht habe, merge ich den PR noch nicht direkt, damit du nochmal schauen kannst, ob es noch in deinem Sinne ist. Sobald du das getan hast, sag Bescheid.
4a3ee39bf5to9bd2b42ba88 months agovielen lieben Dank! Ich habe noch etwas on top implementiert:
das macht die Kostenübersicht leider nochmal etwas länger, insofern würde ich das dann auch demnächst nochmal anfassen.
@ -1278,2 +1279,4 @@else:return 0@propertyDas bedeutet wir erzwingen jetzt, dass LJP Anträge im Kompass erstellt werden, richtig? (Finde ich gut, ich meine mich nur zu erinnern, dass du das mal anders gesehen hast)
@ -119,0 +160,4 @@{% trans "Contributions by the association" %}</td><td>{{ total_subsidies_theoretical }}€Auf was bezieht sich das? Wird aktuell als leer angezeigt, weil keine solche Variable im Kontext existiert. Meinst du
total_subsidies?Wieso werden die LJP Zuschüsse on top zu den übernommenen Belegen ausgezahlt? Zudem werden mehr LJP Zuschüsse als tatsächlich dokumentierte Kosten ausgezahlt, das halte ich für falsch. An erster Stelle die JLs und TNs zu berücksichtigen und erst im letzten Schritt die Kosten des Vereins durch die Zuschüsse gegenzufinanzieren finde ich richtg, aber den JLs mehr Gelder auszuzahlen, als sie Kosten hatten verstehe ich nicht.
@ -383,0 +408,4 @@def paid_ljp_contributions(self):if hasattr(self.excursion, 'ljpproposal') and self.ljp_to:return cvt_to_decimal((1-settings.LJP_TAX) * min(settings.LJP_CONTRIBUTION_PER_DAY * self.excursion.ljp_participant_count * self.excursion.ljp_duration,0.9 * float(self.total_bills_theoretic) + float(self.total_staff)))Ich denke es sollte
0.9 * ( float(self.total_bills_theoretic) + float(self.total_staff))sein. Der LJP macht keinen Unterschied zwischenself.total_bills_theoreticundself.total_staff.sorry, da habe ich etwas unsauber gearbeitet.
Vielen Dank, ziemlich großer Brocken den du da entwickelt hast, sehr cool!
f213e11772into main 8 months agoReviewers
f213e11772.