[Bug 37391] New: QR code for biblio record in OPAC should use canonical link
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Bug ID: 37391 Summary: QR code for biblio record in OPAC should use canonical link Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: OPAC Assignee: schodkowy.omegi-0r@icloud.com Reporter: schodkowy.omegi-0r@icloud.com QA Contact: testopia@bugs.koha-community.org Depends on: 35812 In /opac-detail.pl page there's an option to show a QR code to open up said record on your phone or share it with others in such a way. Currently, it simply uses the current `location.href` of the browser. This isn't ideal, because such URL can be much longer, resulting in more complex/harder to scan QR code, plus URL parameters can contain extra junk, such as our previous search query that led us to current record, which we might not necessarily want to share. Thus, now having simple canonical URLs after the "Depends On" patch was merged in, we can replace these QR codes to link to them instead. Example old URL: https://koha.example.com/cgi-bin/koha/ opac-detail.pl?biblionumber=12345&query_desc=test Example new URL: https://koha.example.com/bib/12345 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35812 [Bug 35812] Should specify canonical URLs to help search indexers -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 --- Comment #1 from Michał <schodkowy.omegi-0r@icloud.com> --- Created attachment 169080 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169080&action=edit Bug 37391: make QR code for biblio record in OPAC use canonical link In /opac-detail.pl page there's an option to show a QR code to open up said record on your phone or share it with others in such a way. Currently, it simply uses the current `location.href` of the browser. This isn't ideal, because such URL can be much longer, resulting in more complex/harder to scan QR code, plus URL parameters can contain extra junk, such as our previous search query that led us to current record, which we might not necessarily want to share. Thus, now having simple canonical URLs after the "Depends On" patch was merged in, we can replace these QR codes to link to them instead. To test: 1. Visit biblio detail page in OPAC 2. Click the button to show QR code in the right pane (preference 'OPACDetailQRCode' must be enabled) 3. Notice that the QR code is much simpler/less complex now 4. Scan the QR code with your phone and confirm it's in the desired format of <URL>/bib/123 and that it opens up properly -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Michał <schodkowy.omegi-0r@icloud.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |januszop@gmail.com, | |roman.dolny@jezuici.pl Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #169080|0 |1 is obsolete| | --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 169082 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169082&action=edit Bug 37391: make QR code for biblio record in OPAC use canonical link In /opac-detail.pl page there's an option to show a QR code to open up said record on your phone or share it with others in such a way. Currently, it simply uses the current `location.href` of the browser. This isn't ideal, because such URL can be much longer, resulting in more complex/harder to scan QR code, plus URL parameters can contain extra junk, such as our previous search query that led us to current record, which we might not necessarily want to share. Thus, now having simple canonical URLs after the "Depends On" patch was merged in, we can replace these QR codes to link to them instead. To test: 1. Visit biblio detail page in OPAC 2. Click the button to show QR code in the right pane (preference 'OPACDetailQRCode' must be enabled) 3. Notice that the QR code is much simpler/less complex now 4. Scan the QR code with your phone and confirm it's in the desired format of <URL>/bib/123 and that it opens up properly Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com QA Contact|testopia@bugs.koha-communit |kyle@bywatersolutions.com |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #169082|0 |1 is obsolete| | --- Comment #3 from Kyle M Hall (khall) <kyle@bywatersolutions.com> --- Created attachment 172992 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172992&action=edit Bug 37391: make QR code for biblio record in OPAC use canonical link In /opac-detail.pl page there's an option to show a QR code to open up said record on your phone or share it with others in such a way. Currently, it simply uses the current `location.href` of the browser. This isn't ideal, because such URL can be much longer, resulting in more complex/harder to scan QR code, plus URL parameters can contain extra junk, such as our previous search query that led us to current record, which we might not necessarily want to share. Thus, now having simple canonical URLs after the "Depends On" patch was merged in, we can replace these QR codes to link to them instead. To test: 1. Visit biblio detail page in OPAC 2. Click the button to show QR code in the right pane (preference 'OPACDetailQRCode' must be enabled) 3. Notice that the QR code is much simpler/less complex now 4. Scan the QR code with your phone and confirm it's in the desired format of <URL>/bib/123 and that it opens up properly Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|QR code for biblio record |QR code for bibliographic |in OPAC should use |record in OPAC should use |canonical link |canonical link -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |24.11.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Pushed for 24.11! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |release-notes-needed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Philip Orr <philip.orr@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This patch updates QR code release notes| |creation from a | |bibliographic record in the | |OPAC to use the canonical | |version of the URL to the | |bibliographic record, | |making the shared URL much | |shorter and the created QR | |code less complicated. Note | |that the system preference | |OPACDetailQRCode must be | |set to 'Enable' in order to | |show the QR code in the | |bibliographic record's | |detail page in the OPAC. Keywords|release-notes-needed | CC| |philip.orr@lmscloud.de --- Comment #5 from Philip Orr <philip.orr@lmscloud.de> --- Added release notes -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37391 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to main |RESOLVED CC| |lucas@bywatersolutions.com Resolution|--- |FIXED --- Comment #6 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Enhancement, no 24.05.x backport -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org