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
[Feature Request] Info about when invoices or quotes were sent to clients #1025
Comments
Those new date fields, I wouldn't place them at the end of the table, but Just a suggestion, otherwise it's going to be a mess with all kinds of fields spread everywhere in those 2 tables. |
Hm, interesting thought. Would be some kind of log then. I'll look into it.
Good point, thx.
Yes, in my current draft I add the new columns after |
@mheiduk, See Feature Request #895. |
@Verony-makesIT thanks! I am aware of this feature, but this led to confusion for my client, because the invoice creation date will also be updated. This is in my opinion a wrong behavior, the creation date should always be the same. If someone wants this feature, it should be at least opt-in. |
@mheiduk, And if your customer is not in favor of this, they can easily change this setting. |
@Verony-makesIT I understand your point, but it's in a legal and also technical perspective not right to do that. A creation date is per definition the date of the creation and should not be altered. |
My client is missing an info about when invoices or quotes were sent to clients. Especially when clients are missing the due date it would be good to know when they were getting a reminder.
Expected Behavior
A date column/field "Sent On" for invoices and quotes.
Current Behavior
No info when invoices or quotes were sent.
Possible Solution/Implementation/Description
New columns in the db tables
invoice_date_sent DATETIME
andquote_date_sent DATETIME
with the info about when invoices were sent to the clients. Will be updated in themark_sent
functions.New column on the invoices/quotes table overview. Furthermore a new datetime field at invoice/quote detail level.
A pull request is already in the making, but before that I have to test it a little bit more. Any further input/suggestion is welcome.
The text was updated successfully, but these errors were encountered: