-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Figgy ARKs resolving to Voyager IDs instead of Alma IDs #1720
Comments
the rake task to generate the figgy_ark_cache errors. The figgy ark cache that is used still has the voyager ids. As a result we build the domain catalog url with the voyager id. This url is included in the electronic_access_1display field and is used in the Available Online section to display the Digital content or Selected images text anchor with this link. The issue is that when the user will click on the link it will not navigate them down to the viewer but it will render a new webpage to resolve the voyager id to the current Alma id. |
Noting the ark_cache we have now is still and using Voyager IDs. |
When pulibrary/figgy#6544 is completed review this ticket again. |
It looks like some of our records in the catalog were indexed with a Figgy cache that still had Voyager IDs instead of Alma IDs.
For example the
electronic_access_1display
field for MMS ID 9956597633506421 was indexed as follow:Notice that it points to an old Voyager ID
5663086
.However if we look at the cache for this record in
bibdata-alma-worker1
it has the new ID (9956630863506421
):This related to #1659
Note that the although the links with the old Voyager IDs do resolve correctly in the browser (i.e. going to https://catalog.princeton.edu/catalog/5663086#view redirects to the proper URL) these Voyager IDs cause problems when using them to find items in Figgy since Figgy does not recognize them anymore. OL makes a GraphQL call to Figgy with the ID in the URL and therefore we must update the cache to get the new Alma IDs in the URL and use those when making the GraphQL call to Figgy. As quick fix I've added a check to OL to bypass this issue (see pulibrary/orangelight#2776) but we should fix the data.
The text was updated successfully, but these errors were encountered: