Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moneybird invoice with no-longer-existing payable #3587

Open
DeD1rk opened this issue Feb 6, 2024 · 1 comment
Open

Moneybird invoice with no-longer-existing payable #3587

DeD1rk opened this issue Feb 6, 2024 · 1 comment
Labels
app:moneybirdsynchronization Issues regarding the Moneybird synchronization bug Something that should be fixed priority: medium A new feature or a bugfix that is non-critical.

Comments

@DeD1rk
Copy link
Member

DeD1rk commented Feb 6, 2024

Describe the bug

There are some invoices whose payable object no longer exists, but do need_synchronization. We need to figure out what to do with that. It shouldn't normally happen (and doesn't happen frequently) but it would be nice if we can either handle it somehow or make it impossible.

How to reproduce

  1. ???

Expected behaviour

???

Additional context

This caused https://thalia.sentry.io/issues/4937025315/.
There is currently one case: invoice 3719, which was for a minimized renewal.

@DeD1rk DeD1rk added priority: medium A new feature or a bugfix that is non-critical. bug Something that should be fixed app:moneybirdsynchronization Issues regarding the Moneybird synchronization labels Feb 6, 2024
@ColonelPhantom
Copy link
Contributor

So I guess this can be reproduced by minimizing an outstanding renewal, or something like that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:moneybirdsynchronization Issues regarding the Moneybird synchronization bug Something that should be fixed priority: medium A new feature or a bugfix that is non-critical.
Projects
None yet
Development

No branches or pull requests

2 participants