Comment 3 for bug 1275118

This looks great Terran! I signed off on your commit and added Erica's signoff to the commit too.

When I was looking at it, I realized that the fulfilled holds also should not display as active since a user would consider those holds to be complete. I added a commit on top of the branch to change those holds to display as fulfilled. In the Active column, then, we would have active holds and three different types of holds users may consider to be no longer active: suspended, canceled and fulfilled.

While I was there, I also changed the "Active On" column header to "Activate" to make it more consistent with the label used in the regular holds interface. I also though the label was a better reflection of what is displaying in that column. New working branch at http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/lp1275118_myopac_canceled_holds

Thank you Terran and Erica!