Comment 11 for bug 1753536

Revision history for this message
Bill Erickson (berick) wrote :

I have confirmed the code works as expected and pushed a new rebase/sign-off branch, including some commit message tweaks on the 2nd commit:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1753536-opac-lib-column-rebase

However, I chose not to merge the code yet. I'm concerned about showing Owning library over Lending library. If the policies may come from either location, and the patron already has an established relationship to the lending library, why not direct them there? Adding another library to the mix seems to be unnecessarily complicating things, more so since it's likely not the branch the patron usually interacts with.

I'm not saying don't merge, but would like some confirmation owning lib is really the choice we want. (Note, I would say add both, but I could also see that being confusing to patrons -- which one do I call?)

Thoughts?