-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add a UI for adding identifiers to authors #853
Comments
@mekarpeles I would like to work on this. Can you please explain a little bit more? |
@ayushshivani The manual editing interface presently allows the user to paste or drag/drop generic html links to these sources. Such links should be automatically recognized and stripped down, to store just the identifier. It should also be possible to simply paste the identifier into a form, including over an existing value. Before storing an id (or deleting a previous value) there should be a check that the target record exists and that it corresponds to the OLID author record (at least one name in common to both records, and non-conflicting lifespan dates, perhaps with some slop allowed). The fetched ISNI records fetched usually include the corresponding VIAF and Wikidata ids. If these do not conflict with an existing value at the OLID target, then they should also be stored there. The ISNI record will provide variant names for the author. These should be added to the OLID target as a.k.a. entries. This will eventually greatly aid author merging. Ultimately much of this could be done as a bot cleanup task, but for now making the manual approach work better will be a great help. Thank you for taking this on. |
@LeadSongDog This ticket has a slightly smaller focus (renamed accordingly). The error checking would be useful, but I think this is just for the first step of adding a UI. Currently, when you edit an edition, you get the option to add identifiers to other sites:
This scope of this ticket is to add a similar interface (ideally re-using as much of this code as possible) to author pages. |
For what it's worth, I just spoke with some editors at Wikimania (a Wikipedia/Wikimedia event) who are interested in contributing corrections to VIAF and Wikidata identifier linkages on Open Library. |
@LeadSongDog That's a good goal, but not the focus of this ticket. Let's avoid scope creep. This ticket is about an identifier edit interface for 3 new author identifiers: Wikidata, VIAF, & ISNI. Other possible, related, future tickets include:
@bnewbold That's excellent, but I think the way that we maximize the efficiency of Wiki* volunteers is to allow them to work in familiar surroundings (ie Wikidata). If we use Wikidata as the central source of truth for all identifiers, we can focus everyone's effort on making sure that is correct. OpenLibrary can then import VIAF, ISNI, Wikipedia, ORCID, etc identifiers/links from there. As a stopgap, in the interim, we'll have the edit interface proposed by this ticket. |
@tfmorris ah, interesting. The way I had envisioned "wikidata as central source of identifier truth" working was that openlibrary entities would have a wikidata QID tag, and then an openlibrary bot/script would use that to do lookups (by wikidata QID), fetch identifiers (and other metadata), then update openlibrary. In this case the onus would be on openlibrary to maintain mappings to wikidata. But I guess you are saying the bot would do lookups based on the openlibrary identifier in wikidata and pull updates based on that. To clarify my above comment, I think the Wiki* editors were really mostly concerned/interested in pushing Wikidata identifier linkage, more than VIAF or other identifiers. Partially, as a mechanism to drive openlibrary visitors through to wikidata (and/or wikipedia articles) as a source of additional information about entities. This isn't possible if the mapping only exists on the wikidata side. I don't have any horse in that race. Anyways, that's all a bit off topic. Regardless what the desired future workflow is, seems important to allow manual correction/editing of identifiers on openlibrary via the web interface. |
I'm adding LCCN and OL to the ones mentioned. Would it be ok for someone to make a list of all the ones mentioned so far? Here it is: |
what is "Classifications: RM666.T65, 616.12061" http://worldcat.org/identities/lccn-n82039404/ is this the VIAF? |
@BrittanyBunk I've updated https://github.com/internetarchive/openlibrary/wiki/Guide-to-Identifiers for your reading pleasure, but it doesn't get into classifications. RM666.T65 is the LC Class for "Theraputics, Vitamin E" while 616.12061 is the Dewey Decimal Class for "Cardiopulmonary Drug Therapies" both of which would reasonably be applied to https://www.worldcat.org/title/vitamin-e-for-ailing-and-healthy-hearts/oclc/488728428 It's easy to tell DDC from LC Classifications, they're all-numeric. |
Is this issue open or is it allotted to somebody as of now? |
@SudarshanHV I accidentally created a duplicate of this issue - so this is still open and an issue. You could tell that it still exists, because the layout for author pages is still a work in progress from when I last checked. You could see the issue I brought up #2786 to find out more. I think you're right though about working on this - so that'll be my next task. |
Hey @BrittanyBunk! 👋 do you need any help with this? |
Yes - the author roles are not alphabetized. It would also be nice if they're categorized. It's also sad that we can't add anymore role names. Why do we need a drop-down menu? If you could remove the drop-down menu and instead let people fill in whatever they need to - that would be ideal! |
Here is the code for how this currently works with editions: openlibrary/openlibrary/templates/books/edit/edition.html Lines 443 to 515 in eddaa12
This is the file to edit to add the new code: https://github.com/internetarchive/openlibrary/blob/eddaa125d1ace42805ea1bc32fcd11d8aeff7603/openlibrary/templates/type/author/edit.html I'm thinking of picking up this ticket and feel like it may be a good use case for Vue (which I am familiar with) since in some future we may want to parse IDs from URLs and giving visual feedback for that is nice. That being said, since #3828 I don't see many (any?) uses of Vue. I'm going to try to follow this doc and see where I get. Any tips, thoughts, or warnings? I'm going to try to keep it simple and just follow this comment with the ability to add/change just three identifiers. |
Related: #144 on the backend -- this added the following fields to Authors, in JSON:
Which are displayed and formatted as links currently on the Author pages, e.g. https://openlibrary.org/authors/OL1102082A under the heading "Links (outside Open Library)"
The missing piece is a field in the Author edit form for users to add these ids.
The text was updated successfully, but these errors were encountered: