KAAV-3388 generateConfirmedFields ei toimi odotetulla tavalla#626
KAAV-3388 generateConfirmedFields ei toimi odotetulla tavalla#626MinnaSiili wants to merge 33 commits intodevelopmentfrom
Conversation
…d fields in projectSaga and generateConfirmedFields
eemeli-kukkonen-siili
left a comment
eemeli-kukkonen-siili
left a comment
There was a problem hiding this comment.
Attribuuttien nimien muodostamisen siirtäminen omiin funktioihin on hyvä muutos joka selkeyttää.
Huomasin, että näiden funktioiden sisältö on vanhaa koodia, joka ei ole koskaan tainnut toimia ihan täsmälleen oikein. Tässä on malliesimerkki siitä, miksi muuttujien nimien parseaminen ja muokkailu on niin ongelmallista varsinkin tässä projektissa jossa muuttujien nimet eivät ole johdonmukaisia.
Unelmoin siitä, että olisi jossain statessa backendistä tallella tarkalleen mikä muuttuja vahvistaa minkäkin päivämäärän, eikä tällaisia stringien leikkailua ja liimailua tarvisi harrastaa ollenkaan. Mutta sitä päivää odotellessa täytyy vaan tarkasti katsoa, että jokaisen muutujan nimi tulee katettua näissä funktioissa.
|
eemeli-kukkonen-siili
left a comment
eemeli-kukkonen-siili
left a comment
There was a problem hiding this comment.
En ole varma oliko tämä vielä kesken kun jira-tiketti oli vielä sulla, mutta kun pyysit reviewta niin seuraavia havaintoja vielä ainakin löytyi:
- generateConfirmedFields ei tarkista vahvistus-kentän arvoa tällä hetkellä ollenkaan, joten on kuin kaikki kentät on "vahvistettu"
- noita vanhentuneita vahvista_paattyy -arvoja ei filtteröidä mitenkään pois
- muutamien elementtijoukkoja, esim. lautakuntia ei oteta mitenkään huomioon
Lisäsin tähän muutaman perustason yksikkötestin, joista voi olla apua
(yarn test)
Katsotaan vielä tarkemmin läpi jos ne saadaan ensin vihreäksi.
…elds are included when confirmed
|
…ing, and special case handling
…ForEsillaoloKey and refactor ehdotus confirmation check
…ing, and ehdotus confirmation keys
…n logic to helper function
…maaraaika filtering, and ehdotus phase special cases
…xity and fix SonarCloud issues
…dAddSpecialCase to reduce cognitive complexity
…Int and globalThis
|
2 participants
Filter deprecated vahvista_paattyy attributes from confirmed fields in projectSaga and generateConfirmedFields