Skip to content
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

Remove default values from publisher entries when pre-processing #621

Open
eporter23 opened this issue Nov 8, 2024 · 4 comments
Open

Remove default values from publisher entries when pre-processing #621

eporter23 opened this issue Nov 8, 2024 · 4 comments
Assignees
Labels

Comments

@eporter23
Copy link
Contributor

eporter23 commented Nov 8, 2024

This ticket follows work to identify why some types of publications are receiving the default "Emory University Libraries" value when an actual publisher entry is present in the descMetadata MODS XML. See initial findings in #565. This seems to be affecting certain non-Article types of publications.

After discussion, we determined the best resolution would be to remove that logic from the processing step.

Desired outcomes:
If a mods:publisher is present using the expected xpath, use that value in the publisher field
If a mods:publisher is not found using the expected xpath, leave publisher blank in the CSV file.

@bwatson78 bwatson78 self-assigned this Nov 12, 2024
@eporter23
Copy link
Contributor Author

For testing, we can generate a new batch of exports and a CSV file in order to see the changes.

@bwatson78
Copy link
Contributor

PR made: #629

@bwatson78
Copy link
Contributor

@bwatson78
Copy link
Contributor

#644

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants