Comment 18 for bug 1676608

Revision history for this message
Galen Charlton (gmc) wrote :

The branch collab/gmcharlt/lp1676608_copy_alerts_rebase_2018_02 has the latest and greatest, consistently of a squash of the previous branch, release notes, and patches to address feedback given during the hackfest. In particular:

- I've tweaked the labeling of the At Circulation and At Ownining Library fields in the copy alert type editor
- Copy alert types are now sorted in modal drop-downs
- Copy alerts are now sorted by ID in the management modal
- I've tweaked the location of the copy alert buttons on the item status detail view
- The Manage button on the item status detail view is now enabled only when there are copy alerts to manage. Also, refreshes are handled better.
- I've fixed a bug in egPCRUD's handling of the apply action

I did not remove the event from the copy alerts acknowledgement dialog, as my notes indicated that that was specifically requested to be there. That said, I'm open to patches to tweak how it styled.

With this, I now ask for a final round of review and merging to master.