finance/statement: show IBAN validity in submit views #104

Merged
christian.merten merged 22 commits from MK/iban_checks into main 11 months ago
Collaborator
  • Add new attribute iban_valid to Member.
  • Submit pages of regular expenses and activity expenses now include an overview which people with expenses have a valid IBAN.
  • Add a note that IBAN validity should be checked manually before submitting.

Right now, no technical barriers are in place that prevent missing/wrong IBANs when a statement is submitted. This is intentional to preserve maximal flexibility.

In a follow-up PR, one could add custom warning messages on top of page when some IBANs are invalid.

* Add new attribute `iban_valid` to `Member`. * Submit pages of regular expenses and activity expenses now include an overview which people with expenses have a valid IBAN. * Add a note that IBAN validity should be checked manually before submitting. Right now, no technical barriers are in place that prevent missing/wrong IBANs when a statement is submitted. This is intentional to preserve maximal flexibility. In a follow-up PR, one could add custom warning messages on top of page when some IBANs are invalid.
marius.klein requested review from christian.merten 11 months ago
christian.merten reviewed 11 months ago
@ -1237,3 +1246,1 @@
"aufgelisteten Kosten korrekt sind und reiche deine Abrechnung dann beim "
"Finanzreferat ein. Wenn du fortschreitest sind keine weiteren Änderungen an "
"der Abrechnung mehr möglich."
"aufgelisteten Kosten korrekt sind, alle notwendigen Belege hochgeladen sind und ob alle Personen, die Geld ausbezahlt bekommen sollen, eine gültige IBAN haben. "

Kannst du hier nochmal makemessages ausführen? Dann wird die zu lange Zeile automatisch umgebrochen.

Kannst du hier nochmal `makemessages` ausführen? Dann wird die zu lange Zeile automatisch umgebrochen.
marius.klein marked this conversation as resolved
@ -17,1 +18,4 @@
return blToColor(klettertreff.has_jugendleiter(jugendleiter))
@register.filter
def render_bool(boolean_value):

Nice! Das fixed auch #69. In einem follow-up könntest du den Tag überall in den Übersichtsseiten verwenden.

Nice! Das fixed auch #69. In einem follow-up könntest du den Tag überall in den Übersichtsseiten verwenden.
Poster
Collaborator

ich hab gar keine Übersichtsseiten gefunden, wo das aktuell falsch dargestellt wird? Kann es aber gerne anpassen.

ich hab gar keine Übersichtsseiten gefunden, wo das aktuell falsch dargestellt wird? Kann es aber gerne anpassen.
Hier zum Beispiel: https://jdav-hd.de/de/kompassfinance/statementsubmitted/7/overview/
christian.merten marked this conversation as resolved
christian.merten reviewed 11 months ago
@ -53,0 +57,4 @@
{{ bill.paid_by.name }}
</td>
<td>
{{ bill.paid_by.iban_valid|render_bool }}

Den valid IBAN check könntest du auch noch unten bei der Auflistung der allowance_to und subsidy_to anzeigen.

Den valid IBAN check könntest du auch noch unten bei der Auflistung der `allowance_to` und `subsidy_to` anzeigen.
Poster
Collaborator

ja auch schon überlegt 👍

ja auch schon überlegt 👍
marius.klein marked this conversation as resolved
marius.klein changed title from WIP: implemented iban checking on submit of expenses to implemented iban checking on submit of expenses 11 months ago
christian.merten reviewed 11 months ago
@ -24,6 +25,35 @@
{% block content %}
<h2>{% translate "Submit to the finance department" %}</h2>
<p>{% translate "Please check if all expenses are documented correctly and if all payers have a valid account code." %}</p>

Man kann auch die Abrechnungen von Freizeiten über "Eingereichte Abrechnungen > Abrechnung einreichen" einreichen. Dann landet man auf dieser Seite und einem werden nur die Auslagen, aber nicht die Aufwandsentschädigungsemfpänger und Zuschussempfänger aufgelistet.

Vielleicht einfach den entsprechenden Abschnitt aus jdav_web/members/templates/admin/freizeit_finance_overview.html hier her kopieren? (gewrapped in {% if statement.excursion %} ... {% endif %}) Ist nicht dringend, bin auch happy das so zu mergen.

Man kann auch die Abrechnungen von Freizeiten über "Eingereichte Abrechnungen > Abrechnung einreichen" einreichen. Dann landet man auf dieser Seite und einem werden nur die Auslagen, aber nicht die Aufwandsentschädigungsemfpänger und Zuschussempfänger aufgelistet. Vielleicht einfach den entsprechenden Abschnitt aus `jdav_web/members/templates/admin/freizeit_finance_overview.html` hier her kopieren? (gewrapped in `{% if statement.excursion %} ... {% endif %}`) Ist nicht dringend, bin auch happy das so zu mergen.
Poster
Collaborator

Alternativvorschlag: wenn eine Abrechnung einer Ausfahrt zugeordnet ist, einfach members/templates/admin/freizeit_finance_overview.html statt finance/templates/admin/submit_statement.html rendern?

Alternativvorschlag: wenn eine Abrechnung einer Ausfahrt zugeordnet ist, einfach `members/templates/admin/freizeit_finance_overview.html` statt `finance/templates/admin/submit_statement.html` rendern?

Finde ich gut, sogar besser.

Finde ich gut, sogar besser.
christian.merten marked this conversation as resolved
christian.merten changed title from implemented iban checking on submit of expenses to finance/statement: show IBAN validity in submit views 11 months ago
christian.merten approved these changes 11 months ago
christian.merten left a comment
Owner

Danke!

Danke!
christian.merten merged commit e7dcbb47ab into main 11 months ago
christian.merten deleted branch MK/iban_checks 11 months ago

Reviewers

christian.merten approved these changes 11 months ago
The pull request has been merged as e7dcbb47ab.
Sign in to join this conversation.
No reviewers
No Milestone
No project
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#104
Loading…
There is no content yet.