[Koha-bugs] [Bug 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Fridolin Somers changed: What|Removed |Added Status|Pushed to main |Needs documenting CC||[email protected] ||m --- Comment #19 from Fridolin Somers --- Enhancement not pushed to 24.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Katrin Fischer changed: What|Removed |Added Text to go in the||This adds the main contact release notes||method recorded in the ||user's account to the table ||of holds awaiting pick-up. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 --- Comment #18 from Katrin Fischer --- Pushed for 25.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Katrin Fischer changed: What|Removed |Added Version(s)||25.05.00 released in|| Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 --- Comment #17 from Katrin Fischer --- (In reply to Owen Leonard from comment #16) > Created attachment 181246 [details] > Interface idea > > Is this too obscure? I like the basic idea, but do we display all the options from the main contact method to be able to put a star on them? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 --- Comment #16 from Owen Leonard --- Created attachment 181246 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181246&action=edit Interface idea Is this too obscure? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Katrin Fischer changed: What|Removed |Added CC||[email protected] Status|Failed QA |Passed QA --- Comment #15 from Katrin Fischer --- Actually, you are right about the accessibility. When I looked at this I thought we had something like: Main contact method: method But we always have below for all cases, including the none case: Main contact method: method So it always represents as one long string. You can't easily hide the "Main contact method", but you could with some JS trickery. I think the "none" case is a bit cluttering as it always adds a lot of text. Same for "Main contact method:" being repeated over and over. Usually the cluttering get worse in German as words tend to be too long and not wrap nicely. To explain my background about changing like this :) I am setting back to PQA and adding Owen. Maybe he has a good idea, otherwise will push as is. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 --- Comment #14 from Baptiste Wojtkowski (bwoj) --- (In reply to Katrin Fischer from comment #13) > Noting a small thing: > > +[% CASE 'fax' %] > +Fax > +[% END %] > + > +[% ELSE %] > +Main contact > method: None > +[% END %] > > I am not sure if we need the None case here. It takes up a lot of space in > the table and could also be deducted from the info now shown. We also lose > the distinction between label and content as it's now both combined into a > string - not sure if that means something for accessibility. > > What do you think? I asked the dev to proceed to the add, I might be wrong but since it is working so for messaging preferences, I would have copied it. If it is still fresh in your mind, how would you deduce from information above ? I think it is important to have the information that nothing is filled, but if you say it can be easily deduced from before at one glance, it is fine for me :) I don't see how it would change accessibility, but I have very few knowledge in that field. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds awaiting pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Katrin Fischer changed: What|Removed |Added Summary|Holds Awaiting Pickup table |Holds awaiting pickup table |should include the patron |should include the patron |main contact method |main contact method Status|Passed QA |Failed QA --- Comment #13 from Katrin Fischer --- Noting a small thing: +[% CASE 'fax' %] +Fax +[% END %] + +[% ELSE %] +Main contact method: None +[% END %] I am not sure if we need the None case here. It takes up a lot of space in the table and could also be deducted from the info now shown. We also lose the distinction between label and content as it's now both combined into a string - not sure if that means something for accessibility. What do you think? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Brendan Lawlor changed: What|Removed |Added Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Baptiste Wojtkowski (bwoj) changed: What|Removed |Added Attachment #179715|0 |1 is obsolete|| --- Comment #12 from Baptiste Wojtkowski (bwoj) --- Created attachment 179716 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179716&action=edit Bug 37860: Add Main contact method to Holds awaiting pickup table This patch adds the main contact method to the patron column of the Holds awaiting pickup table. The style is intended to match the modal on the hold found modal. To test: 1. Apply patch and run yarn build && restart_all 2. Place a hold for a patron and check in the item 3. Notice the modal displays the main contact method in a yellow box 4. Go to Circulation > Holds awaiting pickup 5. Notice the main contact method displays in the Patron column in the same style Signed-off-by: Shi Yao Wang Signed-off-by: Baptiste Wojtkowski Current status: Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Baptiste Wojtkowski (bwoj) changed: What|Removed |Added QA Contact|[email protected] |baptiste.wojtkowski@biblibr |y.org |e.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Baptiste Wojtkowski (bwoj) 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 [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Baptiste Wojtkowski (bwoj) changed: What|Removed |Added Attachment #179700|0 |1 is obsolete|| --- Comment #11 from Baptiste Wojtkowski (bwoj) --- Created attachment 179715 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179715&action=edit Bug 37860: Add Main contact method to Holds awaiting pickup table This patch adds the main contact method to the patron column of the Holds awaiting pickup table. The style is intended to match the modal on the hold found modal. To test: 1. Apply patch and run yarn build && restart_all 2. Place a hold for a patron and check in the item 3. Notice the modal displays the main contact method in a yellow box 4. Go to Circulation > Holds awaiting pickup 5. Notice the main contact method displays in the Patron column in the same style Signed-off-by: Shi Yao Wang Signed-off-by: Baptiste Wojtkowski -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 --- Comment #10 from Brendan Lawlor --- Thanks for looking Baptiste. I added a grey chip to display Main contact method: None -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Brendan Lawlor changed: What|Removed |Added Attachment #176880|0 |1 is obsolete|| --- Comment #9 from Brendan Lawlor --- Created attachment 179700 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179700&action=edit Bug 37860: Add Main contact method to Holds awaiting pickup table This patch adds the main contact method to the patron column of the Holds awaiting pickup table. The style is intended to match the modal on the hold found modal. To test: 1. Apply patch and run yarn build && restart_all 2. Place a hold for a patron and check in the item 3. Notice the modal displays the main contact method in a yellow box 4. Go to Circulation > Holds awaiting pickup 5. Notice the main contact method displays in the Patron column in the same style Signed-off-by: Shi Yao Wang -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Baptiste Wojtkowski (bwoj) changed: What|Removed |Added Status|Signed Off |Failed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Baptiste Wojtkowski (bwoj) changed: What|Removed |Added CC||baptiste.wojtkowski@biblibr ||e.com --- Comment #8 from Baptiste Wojtkowski (bwoj) --- Looks fine to me. Maybe you want to add a gray chip when there is no primary method, as it is for messaging preference ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 --- Comment #7 from Brendan Lawlor --- Thanks Kyle, I appreciate your help. I made those two changes. Adding the switch statement also addresses Shi Yao's commnet #4. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Brendan Lawlor changed: What|Removed |Added Attachment #171998|0 |1 is obsolete|| --- Comment #6 from Brendan Lawlor --- Created attachment 176880 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176880&action=edit Bug 37860: Add Main contact method to Holds awaiting pickup table This patch adds the main contact method to the patron column of the Holds awaiting pickup table. The style is intended to match the modal on the hold found modal. To test: 1. Apply patch and run yarn build && restart_all 2. Place a hold for a patron and check in the item 3. Notice the modal displays the main contact method in a yellow box 4. Go to Circulation > Holds awaiting pickup 5. Notice the main contact method displays in the Patron column in the same style Signed-off-by: Shi Yao Wang -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Brendan Lawlor changed: What|Removed |Added Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Kyle M Hall (khall) changed: What|Removed |Added Status|Signed Off |Failed QA CC||[email protected] --- Comment #5 from Kyle M Hall (khall) --- Couple easy to fix qa issues: 1) primary_contact_method sees be labeled as "Main contact method" in other templates. Can you update that? 2) The primary_contact_method output needs to be run through a an if/elseif or switch statement to it can be translatable. Here is the code from moremember.tt you could use: [% SWITCH patron.primary_contact_method %] [% CASE 'phone' %] Primary phone [% CASE 'phonepro' %] Secondary phone [% CASE 'mobile' %] Other phone [% CASE 'email' %] Primary email [% CASE 'emailpro' %] Secondary email [% CASE 'fax' %] Fax [% END %] -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Shi Yao Wang changed: What|Removed |Added CC||[email protected] Status|Needs Signoff |Signed Off --- Comment #4 from Shi Yao Wang --- Works, but it seems to display the name of the db column as is. E.g. Secondary email shows "emailpro" in step 3 and 5. Maybe we want to display the actual text? Signed-off nonetheless -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Shi Yao Wang changed: What|Removed |Added Attachment #171207|0 |1 is obsolete|| --- Comment #3 from Shi Yao Wang --- Created attachment 171998 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171998&action=edit Bug 37860: Add Main contact method to Holds awaiting pickup table This patch adds the main contact method to the patron column of the Holds awaiting pickup table. The style is intended to match the modal on the hold found modal. To test: 1. Apply patch and run yarn build && restart_all 2. Place a hold for a patron and check in the item 3. Notice the modal displays the main contact method in a yellow box 4. Go to Circulation > Holds awaiting pickup 5. Notice the main contact method displays in the Patron column in the same style Signed-off-by: Shi Yao Wang -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Brendan Lawlor changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 --- Comment #2 from Brendan Lawlor --- Not sure if this is the best way to do the CSS, but the concept is to make the styling match how main contact method is displayed in the hold found modal. See Bug 33398 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 --- Comment #1 from Brendan Lawlor --- Created attachment 171207 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171207&action=edit Bug 37860: Add Main contact method to Holds awaiting pickup table This patch adds the main contact method to the patron column of the Holds awaiting pickup table. The style is intended to match the modal on the hold found modal. To test: 1. Apply patch and run yarn build && restart_all 2. Place a hold for a patron and check in the item 3. Notice the modal displays the main contact method in a yellow box 4. Go to Circulation > Holds awaiting pickup 5. Notice the main contact method displays in the Patron column in the same style -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Lucas Gass changed: What|Removed |Added CC||[email protected] -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 37860] Holds Awaiting Pickup table should include the patron main contact method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37860 Brendan Lawlor changed: What|Removed |Added Assignee|[email protected] |[email protected] |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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/
