search

cul-it / uls

Unified Library Site Development
1 stars 2 forks source link

Check for newcatalog domain on unit sites #1193

Open mhk33 opened 1 year ago

mhk33 commented 1 year ago

And change to catalog domain as needed

mhk33 commented 4 months ago

@sarah-cul It looks like the Mann website search box is searching the newcatalog.library.cornell.edu domain instead of catalog.library.cornell.edu. Can this be easily changed? Would it need to be updated in the central theme and then pushed out to all the units?

Also, ideally content editors should check if they have links to the newcatalog in the content and update them. A quick search for "catalog" on the Mann site and I found a couple of pages such as: https://mann.library.cornell.edu/print-collections https://mann.library.cornell.edu/cornell-agritech

Would it be best to email stakeholders directly and ask them to change the links? Although we have redirects, we probably want to change the URLs if possible.

sarah-cul commented 4 months ago

@mhk33 You are correct, I just found a search.js inside the culu theme that is pointing to newcatalog.library.cornell.edu. Ill make that update for the theme and it can be included in the next upstream effort. thanks for finding that!

As far as content, that would be good to let content owners know, but I'm wondering in the past when our group transitioned to use "newcatalog" was that effort made by us? If so, maybe I should be the one to clean it up?

mhk33 commented 4 months ago

@sarah-cul I updated the links on the main site, but it was mostly because it was just one site and relatively easy for me to do. If you want to do this on the units, I can try to help if we divvy them up. I can understand how that might be easier, overall, than communicating with and waiting for stakeholders to do it themselves.

I think it was relayed to librarians to update their guide links, too, but I have doubts that people did that with older guides.

sarah-cul commented 4 months ago

@mhk33 This might take some time. there are also bit.ly links that refer to newcatalog:

image

Those will need to be replaced with new bit.ly links, or just replaced? Not sure why they are necessary.

I can keep a checklist on this issue, and just try to tackle the sites when possible.

sarah-cul commented 4 months ago

Checklist for ULS:

mhk33 commented 4 months ago

@sarah-cul No idea about the bit.ly links, I wonder who made them. Perhaps a previous developer? Good catch. I'm not sure why we need them either.

Let me know if you want me to help with updating content links!

sarah-cul commented 4 months ago

while going through the sites, I'm checking code snippets and posts by running a terminus command terminus remote:wp uls-site.live -- wp db query SELECT ID, post_title FROM wp_posts WHERE post_content LIKE '%newcatalog.library%' AND post_status like 'publish'

I'm on Asia, there are 100+ instances in the custom post loops like books. Looking into an easy search replace.

sarah-cul commented 3 weeks ago

@mhk33 made some progress this past sprint on these, more than halfway through now!