sde1000 / quicktill

Figure out where all the money and stock went to
GNU General Public License v3.0
43 stars 8 forks source link

"Transaction closed" timestamp? #259

Open sde1000 opened 1 year ago

sde1000 commented 1 year ago

At the moment, transactions have a simple open/closed flag. If we want to know when the transaction was closed we can try to infer it from the date of the most recent transaction line or payment, but we can't be sure. (This may be incorrect in the case of transactions being merged, for example: the merge may result in a closed transaction from multiple open ones, without adding any further translines or payments.)

Some situations have arisen where this would be useful for debugging (eg. "when did this transaction close, and why wasn't it immediately after a payment") and there may be more situations where it's useful when trying to work out what happened.

Would adding a timestamp be sensible? Or would we be better off working on a more general transaction logging system for tracking merges, etc. as discussed in #215?