-
Notifications
You must be signed in to change notification settings - Fork 3
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
86 missing wos ids #108
86 missing wos ids #108
Conversation
We've discovered that some SW pubs could be updated with a new WoSItemID value from SW, so they should be harvested when missing. This script can be enhanced to do so, marked this PR as [WIP] to evaluate this. |
db9a20c
to
e82e1a7
Compare
Running the new script on my laptop prod-db-dump to check it out. The script updated about 30,000 SW publications in the prod-db-dump on my laptop. About 5,000 of those have a new WoSItemID value. |
e82e1a7
to
e1019ca
Compare
This PR looks good. Can you add the scriptie description to https://github.com/sul-dlss/sul_pub/wiki/Useful-Rails-runner-scripties? |
This script will log some details for all the ScienceWire publications with a missing WoSItemID. On a prod-db-dump, all of the identified publications did not have a WoSItemID in the ScienceWireSourceRecord. No attempt is made to re-harvest the publications; if that is a requirement, it can be added, but that will be extra work to change this script from a read-only operation into an update operation.
This should fix #86