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: PDF-generointi hyväksymispäätöksestä lausunnonantajille ja muistuttajille. Koodin refaktorointi helpommin lokalisoitavaksi. #330

Merged
merged 2 commits into from
Aug 24, 2022

Conversation

haapamakim
Copy link
Contributor

No description provided.

@haapamakim haapamakim marked this pull request as draft August 22, 2022 07:12
@ValheKouneli ValheKouneli self-assigned this Aug 22, 2022
@haapamakim haapamakim force-pushed the feature/HASSU-815 branch 3 times, most recently from e4e1b82 to b360be4 Compare August 23, 2022 09:57
@haapamakim haapamakim marked this pull request as ready for review August 23, 2022 09:57
…ä kunnille, lausunnonantajille, muistuttajille. Koodin refaktorointi helpommin lokalisoitavaksi.
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.

Testasin läpi kaikki uudet pdf-esikatselutoiminnot ja tarkistin vanhat esikatselutoiminnot.
Kaikista muista paitsi aloituskuulutuksesen pdf:stä tippuu pois projektipäällikkö "lisätietoja antavat" -listasta, koska frontti ei lähetä sitä tietoa.

Suunnitteluvaiheen kutsun esikatselussa pdf:ään generoituu linkki, jossa mainitaan cloudfront, mikä on epäilyttävää. Tämä saattaa olla siellä vanhastaan, mutta hyvä tarkistaa.

@haapamakim
Copy link
Contributor Author

Testasin läpi kaikki uudet pdf-esikatselutoiminnot ja tarkistin vanhat esikatselutoiminnot. Kaikista muista paitsi aloituskuulutuksesen pdf:stä tippuu pois projektipäällikkö "lisätietoja antavat" -listasta, koska frontti ei lähetä sitä tietoa.

Suunnitteluvaiheen kutsun esikatselussa pdf:ään generoituu linkki, jossa mainitaan cloudfront, mikä on epäilyttävää. Tämä saattaa olla siellä vanhastaan, mutta hyvä tarkistaa.

Siellä oli entuudestaan puutteita. Käyttöliittymässä oli disabloidulla checkboksilla projari ja sunniittelusopimuksen yhteystieto, mutta ne jäivät puuttumaan kutsu-pdf:stä. Nyt ne sinne lisätään kutsussa. Lisäksi projektipäällikkö lisätään nyt aina yhteystietoihin pdf:ssä.

Cloudfront-linkki pdf:ssä koskee vain kehittäjän ympäristöä. Sen korjaaminen olisi ehkä turhan sekavaa näin pientä yksityiskohtaa varten, joten antaisin olla jos ei suurempia tarpeita ilmene.

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.

Tsekattu, ja kaikki toimii. (Huom! Jos projektilla ei ole kuntaa, Nähtävilläolovaiheen Kuulutuksen esikatselu ja Ilmoitus kiinteistön omistajalle esikatselu eivät toimi, mutta tämä ei ole ongelma, koska tulevaisuudessa estetään se, että projekti perustetaan ilman kuntatietoa.)

@ValheKouneli ValheKouneli merged commit 5f4e8ab into main Aug 24, 2022
@ValheKouneli ValheKouneli deleted the feature/HASSU-815 branch August 24, 2022 08:40
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