Add link from holds to hold copy map in fieldmapper

Bug #1845238 reported by Jeff Davis
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Invalid
Wishlist
Unassigned

Bug Description

The fieldmapper should provide a link from holds to hold copy map for reporting purposes. We added a custom link for this at Sitka many years ago to help some of our libraries manage recalls; I think it would be a useful feature for the broader EG community.

Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Working branch user/jeffdavis/lp1845238-holds-link-to-hold-copy-map adds the link to action::hold_request and action::last_hold_request:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=commitdiff;h=e3082a1b

tags: added: holds pullrequest reports
Changed in evergreen:
milestone: none → 3.4-beta2
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

See also bug 1797441.

Changed in evergreen:
milestone: 3.4-beta2 → 3.next
importance: Undecided → Wishlist
Galen Charlton (gmc)
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Terran McCanna (tmccanna) wrote :

I can verify that Jeff's patch updated the fieldmapper cleanly, but I'll leave it to someone more knowledgeable about the field mapper to do the signoff.

Revision history for this message
Terran McCanna (tmccanna) wrote :

I confirmed that the link to the Holds Copy table is now available from the Holds Request table in the reporter. I think this will be useful. My signoff is at:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp1845238-holds-link-to-hold-copy-map-signoff

tags: added: signedoff
Changed in evergreen:
milestone: 3.next → 3.6-beta
Revision history for this message
Galen Charlton (gmc) wrote :

Jeff, upon looking at this - what makes the new link different from the eligible_copies link that was added via commit b3d45facd05ac505b3fde745ac18bf6829a5c4e6 back in 2012? On the face of it, the patch seems redundant.

Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
tags: added: needsdiscussion
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Good catch, Galen, thanks. The commit I shared dates from 2010; I missed the fact that a link to asset.hold_copy_map had been added since then, so my branch is indeed redundant. Sorry for the noise! I'll mark this bug as invalid.

Changed in evergreen:
status: Confirmed → Invalid
Changed in evergreen:
assignee: Galen Charlton (gmc) → nobody
milestone: 3.6-beta → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.