[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #32 from Fridolin Somers --- Enhancement not pushed to 20.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #31 from Jonathan Druart --- Pushed to master for 21.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Jonathan Druart changed: What|Removed |Added Version(s)||21.05.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=28273 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #30 from Tomás Cohen Arazi --- Created attachment 120342 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120342=edit Bug 27864: (QA follow-up) Select2 style on multi-holds Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #29 from Tomás Cohen Arazi --- Created attachment 120341 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120341=edit Bug 27864: (QA follow-up) Hide logged-in option if invalid pickup location Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Status|Failed QA |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Failed QA --- Comment #28 from Jonathan Druart --- I confirm it's directly related to this patch set. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #27 from Jonathan Druart --- Something else (not directly related to this?): 1 - In Circulation and fines rules > Default checkout, hold and return policy, change 'Hold pickup library match' to 'item's home library' 2 - Set AllowHoldPolicyOverride system preference to "*NOT ALLOWED*" 3 - /cgi-bin/koha/reserve/request.pl?biblionumber=233 4 - Search "edna" => https://snipboard.io/F8oLIN.jpg Only Fairview and Midway are available but Centerville is preselected (and not in the dropdown list). Save anyway => Centerville is saved (shouldn't be rejected?) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #26 from Jonathan Druart --- (In reply to Tomás Cohen Arazi from comment #25) > (In reply to Jonathan Druart from comment #24) > > Thanks, Nick. > > > > There is a tiny ergonomic issue with the visual feedback: > > https://snipboard.io/5Wk9IQ.jpg > > Did you try to fix it? > > Hi, how did you get that? I've tried this patches now, on Firefox, Chrome > and Safari and couldn't reproduce the visual glitch. Can be reproduced on FF with zoom at 133% -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Blocks||28261 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28261 [Bug 28261] Add visual feedback on overridden pickup locations on patron's page -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #25 from Tomás Cohen Arazi --- (In reply to Jonathan Druart from comment #24) > Thanks, Nick. > > There is a tiny ergonomic issue with the visual feedback: > https://snipboard.io/5Wk9IQ.jpg > Did you try to fix it? Hi, how did you get that? I've tried this patches now, on Firefox, Chrome and Safari and couldn't reproduce the visual glitch. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Depends on|27931 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27931 [Bug 27931] Add GET /items/:item_id/pickup_locations -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #24 from Jonathan Druart --- Thanks, Nick. There is a tiny ergonomic issue with the visual feedback: https://snipboard.io/5Wk9IQ.jpg Did you try to fix it? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #23 from Tomás Cohen Arazi --- Thank you, Nick! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #22 from Nick Clemens --- Created attachment 120245 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120245=edit Bug 27864: (QA follow-up) Remove size=1 on selects (28066 follow-up) Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #21 from Nick Clemens --- Created attachment 120244 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120244=edit Bug 27864: Correct dropdown size This patch makes the dropdown width inherit the existing style. Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Nick Clemens changed: What|Removed |Added Attachment #120014|0 |1 is obsolete|| Attachment #120015|0 |1 is obsolete|| Attachment #120170|0 |1 is obsolete|| --- Comment #20 from Nick Clemens --- Created attachment 120243 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120243=edit Bug 27864: Visual feedback on overridden pickup locations when placing hold This patch makes the form for placing a hold, use the API to retrieve the valid pickup locations for a biblio. To test: 1 - In Circulation and fines rules > Default checkout, hold and return policy, change 'Hold pickup library match' to 'item's home library' 2 - Set AllowHoldPolicyOverride system preference to Allow 3 - Try placing a hold on a biblio with several items. Ensure that there is one item with homebranch of the patrons branch and one at another branch 4 - Note the pickup location dropdown lists all branches, even those not matching the policy (Item's home library) 5 - FAIL: There's no sign they are overrides to rules 6 - Apply the patch 7 - Repeat 3 8 - SUCCESS: Same behavior as before all libraires listed, but there's a sign + tooltip for overridden ones. 9 - Sign off :-D Note: styling follow-up coming Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Text to go in the||This patch changes the release notes||pickup location dropdown, ||when placing biblio-level ||holds, so it renders a ||sign, alerting that ||circulation rules would be ||overridden if the pickup ||location is chosen. || ||It ||relies on new API routes. Keywords|release-notes-needed| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #19 from Tomás Cohen Arazi --- (In reply to Jonathan Druart from comment #18) > Please provide a full test plan, I don't see the visual feedback. > > > => FAIL: There's no sign they are overrides to rules > > I don't understand what this means. Without the patch, you have a dropdown with the pickup_location=1 branches. With the patch, you have the same [1], but there's a visual sign indicating that choosing some of the pickup locations implies an override to rules. This is affected by circulation rules, so you need to make sure the item homebranch, the patron branch, and the rules make some of the pickup locations invalid (and thus require AllowHoldsPolicyOverride to be picked). [1] The same, but using Select2 and API routes. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #18 from Jonathan Druart --- Please provide a full test plan, I don't see the visual feedback. > => FAIL: There's no sign they are overrides to rules I don't understand what this means. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Jonathan Druart changed: What|Removed |Added Keywords||release-notes-needed -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Status|Failed QA |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #17 from Tomás Cohen Arazi --- Created attachment 120170 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120170=edit Bug 27864: (QA follow-up) Remove size=1 on selects (28066 follow-up) Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #16 from Tomás Cohen Arazi --- (In reply to Jonathan Druart from comment #15) > FAIL koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt >FAIL forbidden patterns > forbidden pattern: Don't use select tag's size attribute > where it is 1 (bug 28066) (line 445) > forbidden pattern: Don't use select tag's size attribute > where it is 1 (bug 28066) (line 448) > > > Do you people forget to run the QA script? I don't get that. I'm pulling the latest qa tools... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Failed QA --- Comment #15 from Jonathan Druart --- FAIL koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt FAIL forbidden patterns forbidden pattern: Don't use select tag's size attribute where it is 1 (bug 28066) (line 445) forbidden pattern: Don't use select tag's size attribute where it is 1 (bug 28066) (line 448) Do you people forget to run the QA script? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |n...@bywatersolutions.com |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Nick Clemens changed: What|Removed |Added Attachment #120012|0 |1 is obsolete|| --- Comment #13 from Nick Clemens --- Created attachment 120014 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120014=edit Bug 27864: Visual feedback on overridden pickup locations when placing hold This patch makes the form for placing a hold, use the API to retrieve the valid pickup locations for a biblio. To test: 1. Try placing a hold on a biblio with several items. => SUCCESS: You get a list of valid pickup locations => FAIL: There's no sign they are overrides to rules 2. Apply this patch 3. Repeat 1 => SUCCESS: Same behavior as before, but there's a sign +tooltip for overridden ones. 4. Sign off :-D Note: styling follow-up coming Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Nick Clemens changed: What|Removed |Added Attachment #120013|0 |1 is obsolete|| --- Comment #14 from Nick Clemens --- Created attachment 120015 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120015=edit Bug 27864: Correct dropdown size This patch makes the dropdown width inherit the existing style. Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Nick Clemens changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Status|Failed QA |Signed Off --- Comment #12 from Tomás Cohen Arazi --- Re-submitted due to some rebase problem. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #11 from Tomás Cohen Arazi --- Created attachment 120013 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120013=edit Bug 27864: Correct dropdown size This patch makes the dropdown width inherit the existing style. Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 --- Comment #10 from Tomás Cohen Arazi --- Created attachment 120012 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120012=edit Bug 27864: Visual feedback on overridden pickup locations when placing hold This patch makes the form for placing a hold, use the API to retrieve the valid pickup locations for a biblio. To test: 1. Try placing a hold on a biblio with several items. => SUCCESS: You get a list of valid pickup locations => FAIL: There's no sign they are overrides to rules 2. Apply this patch 3. Repeat 1 => SUCCESS: Same behavior as before, but there's a sign +tooltip for overridden ones. 4. Sign off :-D Note: styling follow-up coming Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118236|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Attachment #119723|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Attachment #119722|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118232|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added CC||tomasco...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118236|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Attachment #118232|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Nick Clemens changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #9 from Nick Clemens --- The biblio_id and patron_id fields are not populated and the GET request is failing -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #8 from Jonathan Druart --- Nick, could you QA this one? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com Status|Patch doesn't apply |Signed Off --- Comment #7 from Martin Renvoize --- Looks great, Signing off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Martin Renvoize changed: What|Removed |Added Attachment #118236|0 |1 is obsolete|| --- Comment #6 from Martin Renvoize --- Created attachment 119723 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119723=edit Bug 27864: Correct dropdown size This patch makes the dropdown width inherit the existing style. Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Martin Renvoize changed: What|Removed |Added Attachment #118232|0 |1 is obsolete|| --- Comment #5 from Martin Renvoize --- Created attachment 119722 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119722=edit Bug 27864: Visual feedback on overridden pickup locations when placing hold This patch makes the form for placing a hold, use the API to retrieve the valid pickup locations for a biblio. To test: 1. Try placing a hold on a biblio with several items. => SUCCESS: You get a list of valid pickup locations => FAIL: There's no sign they are overrides to rules 2. Apply this patch 3. Repeat 1 => SUCCESS: Same behavior as before, but there's a sign +tooltip for overridden ones. 4. Sign off :-D Note: styling follow-up coming Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 David Nind changed: What|Removed |Added CC||da...@davidnind.com Status|Needs Signoff |Patch doesn't apply --- Comment #4 from David Nind --- Patch no longer applies 8-(.. 118232 - Bug 27864: Visual feedback on overridden pickup locations when placing hold 118236 - Bug 27864: Correct dropdown size Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 27864: Visual feedback on overridden pickup locations when placing hold error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt). error: could not build fake ancestor Patch failed at 0001 Bug 27864: Visual feedback on overridden pickup locations when placing hold Testing notes: - To setup for testing follow the test plan as for bug 27863: . In Circulation and fines rules > Default checkout, hold and return policy, change 'Hold pickup library match' to 'item's home library' . Set AllowHoldPolicyOverride system preference to Allow -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27864] Visual feedback on overridden pickup locations when placing biblio-level hold
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27864 Tomás Cohen Arazi changed: What|Removed |Added Summary|Visual feedback on |Visual feedback on |overridden pickup locations |overridden pickup locations |when placing hold (staff) |when placing biblio-level ||hold -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/