Improve access to library info in OPAC

Bug #1884657 reported by Terran McCanna on 2020-06-22
This bug affects 2 people
Affects Status Importance Assigned to Milestone

Bug Description

When patron is logged in, make their home library info (address, contact info, hours, web site link) more readily available - possibly in the footer and/or in their "My Account" info.

Also, add the ability to view that info for any branch by selecting the branch from a dropdown list. (For example, display the home library contact info, but have a button/link to "Select different library".)

Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Terran McCanna (tmccanna) wrote :

Submitted for review - this adds the library address, email, phone, and website link to the patron account info's Home Library section (where previously there was just the home library's name):;a=shortlog;h=refs/heads/user/mccanna/lp1884657_library_info

tags: added: pullrequest
Terran McCanna (tmccanna) wrote :

Note - I did this on the current OPAC, not based on the upcoming Bootstrap OPAC, so if people like this, we'll need to do it for that version as well.

Terran McCanna (tmccanna) wrote :

Removing pullrequest pending the availability of the Bootstrap OPAC.

Bill Erickson (berick) wrote :

Removing pullrequest tag based on Terran's previous comment.

tags: removed: pullrequest
Jason Boyer (jboyer) wrote :

Hi Terran, since we'll have both available for some time and the bootstrap opac won't be the default in 3.6 I'd encourage you to go ahead with this. So much that I added a followup / signoff here:;a=shortlog;h=refs/heads/user/jboyer/lp1884657_signoff / working/user/jboyer/lp1884657_signoff

I lowered the flesh param back to 1 because it refers to the flesh depth rather than how many tables are specified in the flesh_fields param, ran everything through the html filter, and made the email a live link.

tags: added: pullrequest
Terran McCanna (tmccanna) wrote :

Thanks for taking a look, Jason! I tested your changes and signed off except for the flesh param doesn't work when it is 1, it has to be 2. Maybe the confusion is because the billing_address referenced in the first bit refers to the patron billing_address, but the billing_address in the second param refers to the org unit billing address? My update is here:;a=shortlog;h=refs/heads/user/mccanna/lp1884657_library_info_in_opac_update

(It's also currently loaded onto for testing purposes.)

Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Jason Boyer (jboyer) wrote :

Works for me, Terran. I likely got tripped up on the 1/2 by caching; I had a hard time getting the extra info to appear to begin with.

I've squashed and pushed this to master for 3.6. Thanks!

Changed in evergreen:
status: New → Fix Committed
milestone: none → 3.6-beta
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers