Bug 19494

Summary: Add reservedate to Holds awaiting pickup
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: CirculationAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, katrin.fischer, kyle.m.hall, nick, simon.pouchol
Version: masterKeywords: Academy
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 19494: Add reservedate to Holds awaiting pickup
Bug 19494: Add reservedate to Holds awaiting pickup
Bug 19494: Add reservedate to Holds awaiting pickup
Bug 19494: (QA follow-up) Rename column and apply title-string
Bug 19494: Add reservedate to Holds awaiting pickup
Bug 19494: (QA follow-up) Rename column and apply title-string

Description Marcel de Rooy 2017-10-19 10:00:56 UTC

    
Comment 1 Marcel de Rooy 2017-10-19 10:37:12 UTC
Created attachment 68291 [details] [review]
Bug 19494: Add reservedate to Holds awaiting pickup

Add reservedate to report.
Change label "Available since" to "Waiting since".

Note: The table (already) crosses the boundaries of its container. Also
the vertical spacing for the first column (already) is not the same as the
others. (Adding the second date only makes it more visible.) These minor
design issues should be solved on another report.

Test plan:
[1] Go to Circulation, Holds awaiting pickup
[2] Note the additional column for reservedate.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 2 Simon Pouchol 2017-11-03 11:25:53 UTC
Created attachment 68913 [details] [review]
Bug 19494: Add reservedate to Holds awaiting pickup

Add reservedate to report.
Change label "Available since" to "Waiting since".

Note: The table (already) crosses the boundaries of its container. Also
the vertical spacing for the first column (already) is not the same as the
others. (Adding the second date only makes it more visible.) These minor
design issues should be solved on another report.

Test plan:
[1] Go to Circulation, Holds awaiting pickup
[2] Note the additional column for reservedate.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Simon Pouchol <simon.pouchol@biblibre.com>
Comment 3 Marcel de Rooy 2017-11-06 07:36:58 UTC
(In reply to Simon Pouchol from comment #2)
> Signed-off-by: Simon Pouchol <simon.pouchol@biblibre.com>

Merci beaucoup, Simon.
Comment 4 Marcel de Rooy 2017-12-12 14:54:43 UTC
Created attachment 69738 [details] [review]
Bug 19494: Add reservedate to Holds awaiting pickup

Add reservedate to report.
Change label "Available since" to "Waiting since".

Note: The table (already) crosses the boundaries of its container. Also
the vertical spacing for the first column (already) is not the same as the
others. (Adding the second date only makes it more visible.) These minor
design issues should be solved on another report.

Test plan:
[1] Go to Circulation, Holds awaiting pickup
[2] Note the additional column for reservedate.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Simon Pouchol <simon.pouchol@biblibre.com>
Comment 5 Marcel de Rooy 2017-12-12 14:57:29 UTC
Simple rebase
Comment 6 Katrin Fischer 2017-12-22 00:16:17 UTC
Hi Marcel,

I think <th>Reserve date</th> sould be "Hold date" or "Date hold placed" or similar, as we don't use reserve in the GUI. Also I think you need to add class="title-string" to ensure proper sorting of the date.
Comment 7 Marcel de Rooy 2017-12-22 07:26:44 UTC
Created attachment 70096 [details] [review]
Bug 19494: (QA follow-up) Rename column and apply title-string

As requested by Katrin, the column should not refer to Reserve but Hold.
Sorting should be the same as Waiting since-column.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 8 Katrin Fischer 2017-12-22 10:15:00 UTC
Created attachment 70111 [details] [review]
Bug 19494: Add reservedate to Holds awaiting pickup

Add reservedate to report.
Change label "Available since" to "Waiting since".

Note: The table (already) crosses the boundaries of its container. Also
the vertical spacing for the first column (already) is not the same as the
others. (Adding the second date only makes it more visible.) These minor
design issues should be solved on another report.

Test plan:
[1] Go to Circulation, Holds awaiting pickup
[2] Note the additional column for reservedate.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Simon Pouchol <simon.pouchol@biblibre.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 9 Katrin Fischer 2017-12-22 10:15:04 UTC
Created attachment 70112 [details] [review]
Bug 19494: (QA follow-up) Rename column and apply title-string

As requested by Katrin, the column should not refer to Reserve but Hold.
Sorting should be the same as Waiting since-column.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 10 Jonathan Druart 2017-12-22 16:16:49 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 11 Nick Clemens 2018-01-16 12:13:46 UTC
Enhancement, skipping for 17.11.x.
Awesome work everybody!