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

Metatietoputki20230226 #23

Merged
merged 23 commits into from
Mar 1, 2023
Merged

Metatietoputki20230226 #23

merged 23 commits into from
Mar 1, 2023

Conversation

nvolk
Copy link
Contributor

@nvolk nvolk commented Feb 27, 2023

Korjaa neljä tikettiä (mun pitää käydä vanhemmat tiketit ajatuksella läpi, että onko siellä muutakin avoinna):

MET-232: samanmerkityksellisyyden tarkistus lisätty. (Edellinen versio hyväksyi tietyt kansityyppi vs sidostyyppi -parit, muttei tajunnut hyväksyä erikielisiä versioida kansityypeistä samoiksi. Köh.) Näitä ei edelleenkään käännetä yhtä aktiivisesti kuin jotain X00$e-kenttiä.

MRA-378: $6 TAG-INDEX sallii nyt indeksit, joiden arvo on 100+.

MRA-376: $6-linkkikombot, joissa INDEX on käytetty useampaan kertaan. Nyt pitäisi klaarata tapaukset, joissa INDEX on käytetty monta kertaa, mutta $6:n uniikin TAG-osan avulla. (Testeissä samalla indeksillä kaksi 600-kenttää ja yksi 610-kenttä, 610 saa uuden indeksin, mutta 600:t ei)

MRA-366 (aka MET_209): Lisäsin tuplaosakenttien poiston mergen jälkeisiin vaiheisiin, ja sitten käännösten käsittelyä 040$b:n puuttumisten ja virheiden varalle, mutta joo, vähän menee iffyksi. Nyt ei käänny, jos 040$b swe ja sekä basessa että sourcessa on kirjoittaja, muilla komboilla käännetään. Tässä sellainen logiikka, että kenttä käännetään vain lisäyksen tai mergen jälkeen, mutta nyt kumpaakaan ei tapahtunut.

@nvolk nvolk requested a review from ammsalme February 27, 2023 20:51
@nvolk nvolk requested a review from a team as a code owner February 27, 2023 20:51
return candidateSubfields.find(sf => sf.code === 'q' && isRengaskirja(sf.value));
}


function findCounterpartForCoverType(coverType, candidateSubfields) {
function overrideBingingWithCoverType(coverType, candidateSubfields) {
Copy link
Contributor

@ammsalme ammsalme Mar 1, 2023

Choose a reason for hiding this comment

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

Haluisko tää olla BinDing ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Öö, joo :D

@ammsalme ammsalme merged commit 779365c into dev Mar 1, 2023
nvolk added a commit that referenced this pull request Mar 1, 2023
@ammsalme ammsalme deleted the metatietoputki20230226 branch March 6, 2023 15:02
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