[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Purchase invoices: Import Text?


As part of my efforts to eliminate dead code and write browser tests - even for old code - (as a result of having better coverage testing), I ran into the "Import Text" input box in the Purchase Invoice. The function was added in 2006 in a commit which says "Merge SL-POS".

Now, we've long removed the SL POS, but this input box is still there. It doesn't work though. I've created a branch and PR to remove it. (https://github.com/ledgersmb/LedgerSMB/pull/1350)

Although the commit *says* it's SL POS related, I wonder if it really is, based on the functionality.

Yet the question remains: "Go" to merge the PR?



http://efficito.com -- Hosted accounting and ERP.
Robust and Flexible. No vendor lock-in.
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
Ledger-smb-devel mailing list