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

Do not suppress work titles with source on translations #1328

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

niklasl
Copy link
Member

@niklasl niklasl commented Nov 14, 2023

The rules rely on NormalizeWorkTitles doing its juggling in the background, and cover a bit more than what is currently done by that post-processing.

(Eventually we might come to remove this "source on title blocks bib 240" altogether.)

The rules rely on NormalizeWorkTitles doing its juggling in the
background, and cover a bit more than what is currently done by that
post-processing.

(Eventually we might come to remove this "source on title blocks bib
240" altogether.)
Copy link
Contributor

@klngwll klngwll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:star for documentation! Maybe add the Primary Contribution to the 240 examples and a 130 example just to make sure they both work?

Copy link
Contributor

@kwahlin kwahlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@niklasl
Copy link
Member Author

niklasl commented Nov 15, 2023

:star for documentation! Maybe add the Primary Contribution to the 240 examples and a 130 example just to make sure they both work?

The specs run before all post-processing; so the 130/240 juggling isn't part of the equation. This is a bit obscure, as any advanced post-processing cannot be reflected in the specs (in theory throughout all of marcframe, not just for these two fields).

Not sure what kind of comment is needed (there are many specs in 240 which aren't "real" anymore due to the title "juggling"); this PR already adds:

"IMPORTANT": "Actual results are subsequently modified by the NormalizeWorkTitles postProcessing step!",

which may be enough?

Copy link
Contributor

@klngwll klngwll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@niklasl niklasl merged commit ebfe5b2 into develop Nov 15, 2023
1 check passed
@niklasl niklasl deleted the feature/fmt-201-keep-title-on-translations branch November 15, 2023 16:46
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.

3 participants