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

Met502 kustantajat #111

Merged
merged 11 commits into from
Nov 8, 2023
Merged

Met502 kustantajat #111

merged 11 commits into from
Nov 8, 2023

Conversation

nvolk
Copy link
Contributor

@nvolk nvolk commented Nov 8, 2023

Improvements merging 710$a subfields as per MET-502 and WSOY and ntamo (as per MELINDA-8978).

  • allow field merging prefer "foo (qualifier)" over plain "foo"
  • besides qualifiers support some other "tails" such as "oy" and "kustannusosakeyhtiö"
  • allow field merge for Werner Söderström osakeyhtiö vs WSOY, prefer the longer form
  • prefer ntamo over Ntamo

@nvolk nvolk requested a review from ammsalme November 8, 2023 13:01
@nvolk nvolk requested a review from a team as a code owner November 8, 2023 13:01
@ammsalme ammsalme merged commit 90f5cc3 into next Nov 8, 2023
ammsalme added a commit that referenced this pull request Nov 9, 2023
245punc (#107)

Bump actions/setup-node from 3 to 4 (#102)

Bump the production-dependencies group with 1 update (#103)
* Bumps the production-dependencies group with 1 update: [@natlibfi/marc-record-validators-melinda]

Bump the development-dependencies group with 1 update (#101)
* Bumps the development-dependencies group with 1 update: [eslint](https://github.com/eslint/eslint).

Update field sorting (#105)
* Update dependencies
* Sort fields in postprocessor and fix affected tests

Pair http and https [MET-501] (#106)
* New feature: http vs https difference in subfield value does not prevent field merge [MET-501]

Met502 kustantajat (#111)
* Merge publishers with differing name forms [MET-502]

General
* Update deps
* 2.0.20-alpha.4

---------

Co-authored-by: nvolk <[email protected]>
@ammsalme ammsalme deleted the MET502_kustantajat branch November 9, 2023 11:54
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