Comment 2 for bug 1635354

Some implementation details / questions.

Should we drop the acq.invoice.complete column and replace any references with checks for non-NULL-ness on the new close_date field? Good for data consistency and simplifying code, not so great for having to rebuild reports. (Though, arguably, many such reports would be rebuilt in light of the new columns).

Related, we'd have to back-fill close_date and closed_by values for existing complete invoices during the upgrade. I would favor applying values regardless, but if we drop 'complete' it will be non-negotiable.

Is there precedent for back-filling a user id?