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.

Sign up for GitHub

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

Jump to bottom

Test merged fixes from sheet #137

Open

azaroth42 opened this issue Sep 12, 2024 · 1 comment

Assignees

Labels

bug

Collaborator

azaroth42 commented Sep 12, 2024

doesn't seem to be being applied?

azaroth42 self-assigned this

kkdavis14 assigned kkdavis14 and unassigned azaroth42

Contributor

kkdavis14 commented Jan 7, 2025 •

edited

Loading

I'll take this on as I've been testing this script quite a bit lately. Merged does not seem to be working correctly.

Related bugherd:
https://www.bugherd.com/projects/284041/tasks/2698

This line:

merged |   | Group | http://www.wikidata.org/entity/Q2799943 | /identified_by[classified_as/equivalent/id='http://vocab.getty.edu/aat/300404670']/content[text()='Modemuseum Provincie Antwerpen'] | UPDATE | MoMu - Fashion Museum Antwerp | KKD | YUL following LC, which has outdated form of name. Institution requested we update the record.

is not changing anything. I've tested the Xpath and it should be correct.

kkdavis14 added the bug label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment