svthalia / concrexit

Thalia Website built on Django.
https://thalia.nu
Other
23 stars 12 forks source link

Moneybird contacts are not made for alumni people #3603

Closed T8902 closed 7 months ago

T8902 commented 7 months ago

Describe the bug

People who are alumni don't have a membership anymore even if they still have an account and thus don't get a moneybird contact which is bad.

Expected behaviour

If someone has an account and makes a payment at a thalia event, they should get a moneybird contact.

DeD1rk commented 7 months ago

I discussed with Roel that we can go with an easier/cleaner solution: when a payer exists for an invoice, but we don't have a moneybirdcontact for whatever reason, we can just fall back to using the unknown payer contact id, like we do for invoices without a payer set at all. The project then remains the same, so as far as I know this doesn't really matter for the bookkeeping.

This is more robust and clean because:

@nvoers just to be sure, what do you think about this?