Bug 21070 links the barcode on holds to the item on moredetail.pl. We should make the same change to the hold lists in the staff patron account.
Created attachment 87372 [details] [review] Bug 22633: Link barcodes in hold summary in patron account to moredetail page Bug 21070 links the barcode on holds to the item on moredetail.pl. This patch makes the same change to the hold lists in the staff patron account. To test: - Create some holds, have at least - one item level hold - one hold waiting for pick-up - Take a look at the details and checkouts tabs in the patron account on staff side - Verify that the barcodes link to the record detail page - Apply patch - Verify the barcodes now link to the moredetail page
Created attachment 87725 [details] [review] Bug 22633: Link barcodes in hold summary in patron account to moredetail page Bug 21070 links the barcode on holds to the item on moredetail.pl. This patch makes the same change to the hold lists in the staff patron account. To test: - Create some holds, have at least - one item level hold - one hold waiting for pick-up - Take a look at the details and checkouts tabs in the patron account on staff side - Verify that the barcodes link to the record detail page - Apply patch - Verify the barcodes now link to the moredetail page Signed-off-by: Axel Amghar <axel.amghar@gmail.com>
There's a conflict in the js now
Created attachment 89620 [details] [review] Bug 22633: Link barcodes in hold summary in patron account to moredetail page Bug 21070 links the barcode on holds to the item on moredetail.pl. This patch makes the same change to the hold lists in the staff patron account. To test: - Create some holds, have at least - one item level hold - one hold waiting for pick-up - Take a look at the details and checkouts tabs in the patron account on staff side - Verify that the barcodes link to the record detail page - Apply patch - Verify the barcodes now link to the moredetail page Signed-off-by: Axel Amghar <axel.amghar@gmail.com>
Created attachment 90219 [details] [review] Bug 22633: Link barcodes in hold summary in patron account to moredetail page Bug 21070 links the barcode on holds to the item on moredetail.pl. This patch makes the same change to the hold lists in the staff patron account. To test: - Create some holds, have at least - one item level hold - one hold waiting for pick-up - Take a look at the details and checkouts tabs in the patron account on staff side - Verify that the barcodes link to the record detail page - Apply patch - Verify the barcodes now link to the moredetail page Signed-off-by: Axel Amghar <axel.amghar@gmail.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Nice work! Pushed to master for 19.11.00
Pushed to 19.05.x for 19.05.01