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

[FEATURE-13]: support relators role notes as part of creators text #13

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

davorpa
Copy link
Member

@davorpa davorpa commented Sep 22, 2022

What this PR do?

Add feature(s)

Description

Recently we introduced the ABBR.: note in markdown sources to support relator abbreviations attached to each creator.

e.g. for creators of type translator:

`trl.:` an awesome translator

More relators terms: https://loc.gov/marc/relators/relaterm.html

This PR adds supports to them

Context

I attach two files in a zip: fpb-parser-data.zip. Compares them to test differences. It seems all right

@davorpa davorpa marked this pull request as ready for review September 22, 2022 08:39
index.js Show resolved Hide resolved
index.js Show resolved Hide resolved
index.js Show resolved Hide resolved
e.g. for creators of type translator: `trl.:` ABC
@davorpa
Copy link
Member Author

davorpa commented Sep 26, 2022

Force-push (#13 (comment)) rebase was for integrate #8 functions

@eshellman
Copy link
Contributor

Suppose someone enters
Charles Windsor, a.k.a.: King Charles
or
Megan Thee Stallion
or
Main Author. See also wierd site

@davorpa

This comment was marked as outdated.

@eshellman
Copy link
Contributor

My point is that verifying validity of a relator token is easy.

@davorpa
Copy link
Member Author

davorpa commented Sep 30, 2022

My point is that verifying validity of a relator token is easy.

Then, I will also put a list like languages.js defining all of them. Thanks for the suggestions

@davorpa davorpa marked this pull request as draft September 30, 2022 15:24
This flag validates the relator term and it position inside each creator string
@davorpa davorpa marked this pull request as ready for review October 1, 2022 18:06
@davorpa
Copy link
Member Author

davorpa commented Oct 1, 2022

My point is that verifying validity of a relator token is easy.

Done

@davorpa davorpa requested a review from eshellman October 1, 2022 18:08
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.

2 participants