finance/statement: make subsidy_to and allowance_to optional #112

closes #108

fields for receivers of allowance a and subsidies s are not compulsory anymore. If receivers are given, a and s are payed out for them. s can only be payed out in total, disregarding the number of receivers given for a.
In the overview pages (activity and statement), we are now distingushing between theoretical a + s and what has been payed out practically. If no receiver was given for either things, it is made clear that nothing is payed out. All combinations of 0, ..., n payed allowances (n being number of approved participating YLs) and 0, 1` payed subsidies are possible.
transaction generation logic adapted accordingly

more todos:

adapt test logic to new transaction logic

closes #108 - fields for receivers of allowance `a` and subsidies `s` are not compulsory anymore. If receivers are given, `a` and `s` are payed out for them. `s` can only be payed out in total, disregarding the number of receivers given for `a`. - In the overview pages (activity and statement), we are now distingushing between theoretical `a` + `s` and what has been payed out practically. If no receiver was given for either things, it is made clear that nothing is payed out. All combinations of `0`, ..., `n` payed allowances (`n` being number of approved participating YLs) and `0, `1` payed subsidies are possible. - transaction generation logic adapted accordingly more todos: - [x] adapt test logic to new transaction logic

Danke! Sieht gut aus, ich würde noch ein paar Tests hinzufügen. Möchtest du das selbst machen oder soll ich einfach auf den branch hier pushen?

Habe auf main ein paar mehr Tests für finance/models.py gepushed. Einer ist mit skip markiert, wenn du main in diesen branch mergest, solltest du @skip entfernen können und der Test sollte funktionieren.

Habe auf `main` ein paar mehr Tests für `finance/models.py` gepushed. Einer ist mit `skip` markiert, wenn du `main` in diesen branch mergest, solltest du `@skip` entfernen können und der Test sollte funktionieren.

Von mir aus siehts gut aus. Was meinst du mit

more todos:
[ ] adapt test logic to new transaction logic

Von mir aus siehts gut aus. Was meinst du mit > more todos: > [ ] adapt test logic to new transaction logic

Von mir aus siehts gut aus. Was meinst du mit

more todos:
[ ] adapt test logic to new transaction logic

dadurch, dass sich ja die Logik geändert, welche Beträge ausgezahlt werden, wollte ich da nochmal reingucken. Hatte aber bisher keine Zeit, mich da reinzufuchsen... Verstehe ich das richtig, dass du da schon was gemacht hast? Habs gerade nochmal auf testing gepushed und würde es da nochmal final testen. Dann wäre ich ready zum mergen.

> Von mir aus siehts gut aus. Was meinst du mit > > more todos: > > [ ] adapt test logic to new transaction logic dadurch, dass sich ja die Logik geändert, welche Beträge ausgezahlt werden, wollte ich da nochmal reingucken. Hatte aber bisher keine Zeit, mich da reinzufuchsen... Verstehe ich das richtig, dass du da schon was gemacht hast? Habs gerade nochmal auf testing gepushed und würde es da nochmal final testen. Dann wäre ich ready zum mergen.

Von mir aus siehts gut aus. Was meinst du mit

more todos:
[ ] adapt test logic to new transaction logic

dadurch, dass sich ja die Logik geändert, welche Beträge ausgezahlt werden, wollte ich da nochmal reingucken. Hatte aber bisher keine Zeit, mich da reinzufuchsen... Verstehe ich das richtig, dass du da schon was gemacht hast? Habs gerade nochmal auf testing gepushed und würde es da nochmal final testen. Dann wäre ich ready zum mergen.

Wenn du mit "testing logic", das meinst was bei make test abgefragt wird, dann ja. Das ist angepasst und die Tests gehen alle durch. Hast du deinen finalen Test gemacht?

> > Von mir aus siehts gut aus. Was meinst du mit > > > more todos: > > > [ ] adapt test logic to new transaction logic > > dadurch, dass sich ja die Logik geändert, welche Beträge ausgezahlt werden, wollte ich da nochmal reingucken. Hatte aber bisher keine Zeit, mich da reinzufuchsen... Verstehe ich das richtig, dass du da schon was gemacht hast? Habs gerade nochmal auf testing gepushed und würde es da nochmal final testen. Dann wäre ich ready zum mergen. Wenn du mit "testing logic", das meinst was bei `make test` abgefragt wird, dann ja. Das ist angepasst und die Tests gehen alle durch. Hast du deinen finalen Test gemacht?

Labels Milestones

finance/statement: make `subsidy_to` and `allowance_to` optional #112

Reviewers

Labels Milestones

finance/statement: make subsidy_to and allowance_to optional #112

Reviewers

finance/statement: make `subsidy_to` and `allowance_to` optional #112