feat: hassu 1161, kunnan edustajan poiston esto #542
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 9 commits
January 17, 2023 13:43
…ncin yhteydessä, jos varahenkilö on kunnan edustaja, ja varahenkilö vaihtuu
…ois rikkinäiset henkilöreferenssit
…toja tai kuulutusYhteystietoja, mutta näissä testeissä ei näemmä oo tarkastettu, että referoidut henkilöt on myös kayttoOikeuksissa. Nyt ne siis filtrautuu pois.
ValheKouneli
force-pushed
the
feature/HASSU-1161
branch
from
0e2c6d0
to
925cce1
Compare
|
Mun mielestä näyttää hyvältä... Mutta haluaisin kuulla miksi noi snapshotit on muuttunut. Miksi se käyttäjälista on tyhjänä? Onko bugi vai meneekö oikein? |
|
Tkork: snapshotit on muuttuneet, koska käyttäjäreferensseistä filtrataan pois henkilöt, jotka puuttuvat projektin henkilöistä. Tällä vältetään rikkinäiset referenssit. |
mustonenk
pushed a commit
that referenced
this pull request
Jan 23, 2023
* Estä UI:ssa suunnittelusopimuksen kunnan edustajan poistaminen * Kunnan edustajan poistaminen on estetty BE:ssä. * Lisää testejä KayttoOikeudetManageriin * Projekti handler test sille, ettei kunnan edustajaa poisteta velho syncin yhteydessä * Projekti handler test sille, ettei kunnan edustajaa poisteta velho syncin yhteydessä, jos varahenkilö on kunnan edustaja, ja varahenkilö vaihtuu * Korjaa testin kuvaus ja kommentti * Päivitä adaptToAPI (virkamies) adaptereita niin, että ne filtraavat pois rikkinäiset henkilöreferenssit * Luo testejä sille, että adapterit filtraavat pois rikkinäiset henkilöreferenssit * Päivitä snapshotit. Oikeasti ei voi olla tyhjiä esitettäviä yhteystietoja tai kuulutusYhteystietoja, mutta näissä testeissä ei näemmä oo tarkastettu, että referoidut henkilöt on myös kayttoOikeuksissa. Nyt ne siis filtrautuu pois. Co-authored-by: Valhe Kouneli <valhe.kouneli@cgi.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Toteutettu bäkkäriin ja fronttiin esto kunnan edustajan poistamiselle. Jos projektin henkilö on kunnan edustaja, sille ei pitäisi näkyä UI:ssa poistonappulaa. Lisäksi BE estää kunnan edustajan poistamisen.
Velho-synkronoinnin yhteydessä kunnan edustaja ei poistu, vaikka kunnan edustaja olisi ollut velhosta alun perin tullut projektin projektipäällikkö tai varahenkilö. Jos velhosta tulee synkronoinnin yhteydessä uusi projektipäällikkö tai varahenkilö, kunnan edustaja muutetaan tavalliseksi, muokattavissa olevaksi projektin henkilöksi synkronoinnin yhteydessä.
Kunnan edustaja on mahdollista vaihtaa, minkä jälkeen vanhan kunnan edustajan voi poistaa. Aloituskuulutuksessa, VuorovaikutusKierroksessa, NahtavillaoloVaiheessa, HyvaksymisPaatosVaiheessa ja JatkoPaatosVaiheissa voi olla tämän jälkeen yhä referenssejä vanhaan kunnan edustajaan. Tämä ratkaistaan sillä, että virkamiespuolen adapterit filtteröivät rikkinäiset referenssit pois. Ei ole mahdollista tulla tilannetta, jossa UI:ssa näkyy rikkinäinen refenressi. Sen sijaan kaikkiin julkaistuihin materiaaleihin on muutettu referenssi yhteystiedoksi.
Testausvaiheessa on hyvä testata mm. se, että luo projektin, jolla on velhosta tulevien henkilöiden lisäksi joku kunnan edustaja. Sitten luo vuorovaikutuskierroksen soittoajalla, jonka yhteystieto on kunnan edustaja (ei muita). Sitten vaihtaa kunnan edustajan toiseksi, poistaa alkuperäisen kunnan edustajan ja menee muokkaamaan soittoajan yhteystietoja. Muokkaustilaan pitäisi päästä, mutta siinä ei pitäisi näkyä yhtään valittua yhteystietoa. Ennen tallennusta on annettava jokin uusi yhteystieto.