Skip to content

Use alternate_names fully for existing Author matching#11923

Merged
mekarpeles merged 3 commits intointernetarchive:masterfrom
hornc:transliterate
Mar 6, 2026
Merged

Use alternate_names fully for existing Author matching#11923
mekarpeles merged 3 commits intointernetarchive:masterfrom
hornc:transliterate

Conversation

@hornc
Copy link
Copy Markdown
Collaborator

@hornc hornc commented Feb 24, 2026

alternate_names is being extracted from the MARC record, it appears it might not be recognised by the
create-new-author code? No, alternate_names were NOT being imported #11927 corrects this

Closes #11877

  • Uses incoming record's alternate_names in the name queries used to find existing Author records by name.
  • Comma flips alternate_names to natural name order, as we do with other names

Technical

Testing

Screenshot

Stakeholders

Copilot AI review requested due to automatic review settings February 24, 2026 23:33

This comment was marked as outdated.

@hornc hornc marked this pull request as draft February 24, 2026 23:57
@hornc hornc marked this pull request as ready for review March 4, 2026 01:40
@hornc hornc changed the title WIP: Are author alternate_names being imported? Are author alternate_names being imported? Mar 4, 2026
@hornc hornc changed the title Are author alternate_names being imported? Use alternate_names fully for existing Author matching Mar 4, 2026
@mekarpeles mekarpeles merged commit 35fdb69 into internetarchive:master Mar 6, 2026
4 checks passed
@mekarpeles mekarpeles self-assigned this Mar 6, 2026
@hornc hornc deleted the transliterate branch March 8, 2026 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Transliterated author names not being imported (or matched)

3 participants