feat: Automaatio aineistojen kategorisoinnin lisäykseen #536
Conversation
There was a problem hiding this comment.
Tarvitsee lisää testejä.
Lisäksi ei toimi kuten oli tarkoitus, eli jos UI:ssa valitsee "Tuo aineistoja" Suunnitteluaineistojen alla, valitsee ison kasan tiedostoja ja klikkaa "Tuo aineistot", tällä hetkellä aineistoja menee myös muiden pääkategorioiden alle.
Jatkokehitykseen tulee mieleen useampikin tiketti, mutta jos tuon nyt korjaisi ja tekisi testit, olisi edes sisäisesti eheästi toimiva kokonaisuus.
tkork
force-pushed
the
feature/HASSU-976
branch
from
084af3f
to
68a855b
Compare
|
tkork
force-pushed
the
feature/HASSU-976
branch
2 times, most recently
from
6e835ca
to
06da570
Compare
There was a problem hiding this comment.
Kun lisäsin nähtävilläolovaiheelle aineistot, mutta en lisäaineistoja, ja sitten painoin tallenna ja siirry..., tuli Unhandled Runtime Error
TypeError: Cannot read properties of null (reading 'aineistotValmiit')
Tiedostossa src/hooks/useProjektinTila.ts rivillä 15
if (tila.aineistotValmiit) {
toi tila ei ole määritelty...
| }); | ||
| aineistot | ||
| .filter((velhoAineisto) => !find(value, { dokumenttiOid: velhoAineisto.oid })) | ||
| .map<AineistoInput>((velhoAineisto) => ({ dokumenttiOid: velhoAineisto.oid, nimi: velhoAineisto.tiedosto })); |
There was a problem hiding this comment.
Täältä puuttuu toi forEach((aineisto) => { value.push(aineisto) }
There was a problem hiding this comment.
Toi ylemmän kommentin havainto johtuu mainissa jo olevasta ongelmasta, ei tässä featuresta tehdyistä muutoksista. Sinne korjattiin tänään se ongelma.
Hyvä havainto tämä toinen. Korjaan, testailen vielä vähän ja rebasetan tän.
… ne on Velhossakin
tkork
force-pushed
the
feature/HASSU-976
branch
from
06da570
to
25a387d
Compare
There was a problem hiding this comment.
Valitettavasti suunnitteluvaiheen aineistoissa on vieläkin virheitä. Jos lähtötilanne on se, että on hassu automaattitestiprojekti, ja sillä on 3 esittelyaineistoa, ja sitten valitsee jotkin 3 muuta tiedostoa suunnitelmaluonnoksiksi, niin UI:ssa näkyy kaikki oikein, mutta tallentamisen jälkeen esittelyaineistot ovat hävinneet (UI:ssa niitä ei ainakaan käy).
Myös toinen bugi tuli vastaan. Siinä oli lähtötilanteessa tiedostoja sekä esittelyaineistoissa että suunnitelmaluonnoksissa. Kun suunnitelmaluonnokset poisti ja tallensi, tallentamisen jälkeen kaikki aineistot olivat hävinneet.
|
En saanut toistettua mitään mainitsemistasi ongelmista. Enkä nyt käsitä missä kohtaa olisin rikkonut suunnitteluvaiheesta noita toiminnallisuuksia. Jos siellä on jotain hajalla, pitäisi tarkistaa ensin onko ongelma myös mainissa. Jos on, lisätään bugitikettiä ja korjataan niillä. Jos taas ei ole, kaipaisin toisto-ohjeita ja näyttöä siitä, että minä olen hajottanut toiminnallisuudet näillä muutoksilla. |
No description provided.