-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add status update batch action to invoices table #3756
Conversation
ecd8260
to
c369f43
Compare
c4da0a1
to
3bafd45
Compare
3bafd45
to
b793e3c
Compare
In testing when there were two invoices that had the status
At activity_feed.py. More info in sentry but it still seems to update the status. Seemed also specific to these statuses as others would update w/o issue. This was done as the finance role. |
b793e3c
to
4f4aed7
Compare
@frjo Its latest version is ready to put on test. Although this was working fine for invoices with invoice_number so it wouldn't make any issue on prod still I have added a check to resolve this issue for older invoices as well. |
For some reason now whenever I go to
Which is weird, was there an issue with |
4f4aed7
to
bc2e20f
Compare
@wes-otf I missed doing a rebase for this PR, we have recently flatten the dir structure in another PR. Should be fixed now. |
Description
Fixes part of #3537
Test Steps
paid
an invoice that is notApproved by finance
yet.