Comment 11 for bug 1743801

Revision history for this message
Kathy Lussier (klussier) wrote :

Thank you for the code contribution idjit! It works for me, and I've pushed a signoff to the working repo at:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/lp1743801-copy-status-columns-signoff

I held off on merging it because I would like a second opinion on the way you handled the loan duration and the fine level. The cause for my hesitation was that future development (or possible configuration?) could expand the number of duration and fine levels, but this assumes that there are three distinct levels.

On the other hand, we're already doing something very similar in current Evergreen in the item status full detail view, so it may be perfectly fine. Does anyone else have thoughts on this?