-
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 invoice date and paid date to invoice and invoices lists and tables #4019
Conversation
Migration conflict with another PR on test so we wait to test this for a bit. |
41652b7
to
47bcb6d
Compare
47bcb6d
to
1372124
Compare
Just had a demo meeting with finance and they liked it! When the pay date is set via the update status modal, where is that info displayed? I couldn't find where that was listed after it was set. |
two other quick notes: |
hypha/hypha/apply/projects/templates/application_projects/includes/invoices.html Lines 47 to 53 in 1372124
|
@@ -182,6 +182,8 @@ class Invoice(models.Model): | |||
null=True, | |||
verbose_name=_("Invoice amount"), | |||
) | |||
invoice_date = models.DateTimeField(null=True, verbose_name=_("Invoice date")) | |||
paid_date = models.DateTimeField(null=True, verbose_name=_("Paid date")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can use DateField
for both these fields.
Fixes #3537
Invoice date:
Paid Date:
Test Steps