Vervollständigung des Finanz-Workflows
#141
Closed
opened 9 months ago by marius.klein
·
12 comments
MK/finance_tests
mk-personal-profile
main
testing
MK/conditional_fields
MK/meeting_checklist
all-inclusive
MK/finance_workflow
cm-oauth
MK/pdf_pagesize
cm-bump-4.2
cm-remove-jet
docu-new-group
toml-configuration-with-templates
individual-sender-address
changes-ku
authentik-integration
jenkins
object-level-permissions
dev
clubdesk-migration
docker
v1-0-stable
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#141
Reference in New Issue
There is no content yet.
Delete Branch '%!s(<nil>)'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
Folgende Punkte sind im Gespräch mit Fabrizio zum Workflow der Ausfahrtenabrechnung und Auslagerückerstattung aufgekommen:
Bei Zusammenfassung von Überweisungen wäre es hilfreich, wenn auch die Einzelbeträge im zusammengefassten Verwendungszweck auftauchen, das erleichtert die Zuordnung von Belegen für die Buchhaltung. Gleichzeitig darf ein Verwendungszweck maximal 140 Zeichen haben.
Ich habe noch keine Ahnung, wie wir die Auszahlung von Zuschüssen ggf. für die Buchhaltung mit Belegen unterfüttern müssen. Ich meine Robert hat damals einfach die unterschriebenen Zuschussantrag-PDFs weitergeleitet, die aus dem Excel generiert wurden. Ich würde da mit Fabrizio auf die GS zugehen, wie wir das möglichst einfach lösen.
in Zusammenhang mit dem zweiten Punkt ist denke ich auch #92 relevant, da hat Robert schon einen Ausfahrten-Beleg beschrieben
außerdem: mir ist beim Vergleich mit dem alten Excel-Formular aufgefallen, dass wir im Kompass das Thema Organisationsbeitrag ab 27J noch nicht abbilden. Nach meinem Verständnis wird pro TN >=27 ein Beitrag von 10€ pro Tourentag erhoben. Der Gesamtbetrag wird von den Zuschüssen abgezogen und muss von den Organisator:innen umgelegt werden.
Dein PR deckt das soweit ich verstehe alles ab, richtig? Noch ein Gedanke bzw. eine Frage für Fabrizio: Können wir den Beleg automatisch / auf Knopfdruck verschicken? Dann ist kein Runterladen + Email verschicken notwendig. Oder schreibt Fabrizio typischerweise einen angepassten Text in der E-Mail?
weitere Punkte aus Gespräch mit Fabrizio und Samuel:
Auszahlung LJP-Geld (Feld für Empfänger)
Berechnung von LJP-Geld (nur 90% in Kostenübersicht anzeigen)
Versenden des Belegs an GS kann automatisiert werden.
ich setze mich die Tage nochmal dran
bis auf den automatischen Versand an die GS ist hier mMn alles erledigt.
Es wäre wunderbar, wenn wir noch ein paar Tests für die ganzen neuen Finanzregeln anlegen.
Re: automatischer Versand: Genügt ein weiterer Knopf "Bestätigen und Beleg an Geschäftsstelle verschicken" in der "Bitte führe die folgenden Überweisungen aus .." Seite?
ja ich denke das wäre perfekt
Ist erledigt in
0eedc3ecf9. Die ausführende Person bekommt die Mail im CC.Fabrizio darf gerne Verbesserungsvorschläge am E-Mail Text / Ablauf vorschlagen. Ansosten den Issue schließen.
vielen Dank dir! Ich hab mich mal noch an ein paar Tests versucht, was hältst du davon? (branch MK/finance_tests)
Sieht gut aus, wenn du einen PR machst habe ich ein paar wenige Kommentare. Ich habe soeben ein bisschen die
memberstests umorganisiert, insbesondere habe ich zwei utility funktionen inmembers/tests/utils.pyextrahiert. Die könntest du in deinen tests benutzen, um die TNs der Ausfahrt zu generieren.Jetzt wo die tests gemerged sind, schließe ich diesen PR mal. Falls noch was fehlt, bitte neuen aufmachen.