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

feat: nahtavillaolo kansalaisnakyma (HASSU-745) #269

Merged
merged 2 commits into from
Jun 14, 2022

Conversation

kettunju
Copy link
Contributor

sis. backendia ja APIa siltä osin, että pystyi testaamaan - backend tarvitsee jatkokehitystä esim julkisen nahtavillaolon lopulliset kentät, tuleeko snapshot malli, adapterin toiminta jne

@kettunju
Copy link
Contributor Author

testausta varten nahtavillaolo taytyy kayda kannasta merkitsemassa tilaan HYVAKSYTTY ja jos jotain vaadittavia kenttia ei viela ole yllapitopuolella, niin nekin kantaan tai vaikka suoraan APIsta graphql ide:lla esim:
mutation myNahtavillaolo {
tallennaProjekti(
projekti: {
oid: "1.2.246.578.5.1.2746809348.1809406873"
nahtavillaoloVaihe: {
hankkeenKuvaus: {
RUOTSI: "Syftet med fasen är att på ett naturligt sätt koppla nuvarande och framtida markanvändning till Tavastehusvägen, att ta hänsyn till områdets bullerskydd, att förbättra flödet och säkerheten för passagerare och kollektivtrafik samt att göra gång- och cykelförbindelserna smidiga. och säker. Att förbättra flödet av tung trafik och förutsägbarheten i restid är också ett av målen."
SUOMI: "Nähtävilläolovaiheen tavoitteena on nykyisen ja tulevan maankäytön liittäminen luontevasti Hämeenlinnanväylään, huomioida alueen melunsuojaus, parantaa henkilöautoliikenteen ja joukkoliikenteen sujuvuutta ja turvallisuutta sekä tehdä jalankulun ja pyöräilyn yhteydet sujuviksi ja turvallisiksi. Raskaan liikenteen sujuvuuden ja matka-ajan ennustettavuuden parantaminen on myös yksi tavoitteista."
}
kuulutusPaiva: "2022-06-13T13:01"
kuulutusVaihePaattyyPaiva: "2022-07-14T13:01"
muistutusoikeusPaattyyPaiva: "2022-07-14T13:01"
kuulutusYhteysHenkilot: ["A014044", "LX511881", "L036511"]
kuulutusYhteystiedot: [
{
etunimi: "Ulla"
sukunimi: "Uusi"
organisaatio: "Ramboll"
puhelinnumero: "029123123"
sahkoposti: "ulla.uusi@rambo.ll"
titteli: "DI"
}
]
}
}
)
}

Copy link
Contributor

@haapamakim haapamakim left a comment

Choose a reason for hiding this comment

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

Puolestani ok näin. Tuunataan paremmaksi kun saadaan tilanhallinta ylläpitopuolella kuntoon.

@kettunju kettunju force-pushed the feature/HASSU-745-nahtavillaolo-kansalaisnakyma branch from a07d7ce to fe62c6b Compare June 14, 2022 10:15
Copy link
Contributor

@ValheKouneli ValheKouneli left a comment

Choose a reason for hiding this comment

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

Tämä lienee hyvä nyt toistaiseksi. Sitten kun Mikko tekee BE-juttunsa, pitää vaan katsoa, että merge menee nätisti. Tässä tehtyjä juttuja pitää silloin päivittää. Muistetaan vaan tehdä se!

<h4 className="vayla-small-title">{t(`ui-otsikot.nahtavillaolo.yhteystiedot`)}</h4>
<SectionContent>
<p>
{t("common:lisatietoja_antavat", {
Copy link
Contributor

Choose a reason for hiding this comment

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

Tässä voisi olla otettu huomioon se, että yhteystietoja onkin vain yksi, eli olisi erikseen "yhteystietoja_antaa" ja "yhteystietoja_antavat"

: t(`common:ely-keskukselle`)}{" "}
{t(`info.nahtavillaolo.ei-rata.ennen_paattymista`)}{" "}
<HassuLink href={window.location.href}>{window.location.href}</HassuLink>.{" "}
{t(`info.nahtavillaolo.ei-rata.sahkopostilla_muistutus`)}
Copy link
Contributor

Choose a reason for hiding this comment

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

Tässä voisi ottaa huomioon sen, että eri kielissä rakenteet ovat tosi erilaisia, ja sen sijaan että pakottaisi sen, mihin kohtaan lausetta linkki tulee, niin antaisikin käännöksen, jossa on jonkinlainen placeholder, joka sitten replacetaan varsinaisella linkillä tässä.

}
return true;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Tätä pitää toki sitten muuttaa, kun/jos Mikko tekee nähtävilläolovaiheesta samanlaisen kuin aloituskuulutusvaiheesta, eli tulee ne "julkaisut"

const { t } = useTranslation("projekti");
const oid = typeof router.query.oid === "string" ? router.query.oid : undefined;
const { data: projekti } = useProjektiJulkinen(oid);
const kuulutus = projekti?.nahtavillaoloVaihe;
Copy link
Contributor

Choose a reason for hiding this comment

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

Jos Mikko tekee nähtävilläolosta sellaisen kuin aloituskuulutuksesta, tähän täytynee tulla tyyliin
const kuulutus = projekti?.nahtavillaoloVaihe?.nahtavillaoloVaiheJulkaisu[projekti?.nahtavillaoloVaihe?.nahtavillaoloVaiheJulkaisu.length-1];

@kettunju kettunju merged commit 325b8d9 into main Jun 14, 2022
@kettunju kettunju deleted the feature/HASSU-745-nahtavillaolo-kansalaisnakyma branch June 14, 2022 10:20
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.

3 participants