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.

Sign up for GitHub

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

Jump to bottom

fix: HASSUYP-289 Lisää paikat saamenkielisille kiinteistönomistajien ja muistuttajien kirjeille nähtävilläoloon ja hyväksymispäätökseen #1292

Merged
merged 12 commits into from
Sep 19, 2024
Merged

fix: HASSUYP-289 Lisää paikat saamenkielisille kiinteistönomistajien ja muistuttajien kirjeille nähtävilläoloon ja hyväksymispäätökseen #1292

merged 12 commits into from
Sep 19, 2024

Conversation

tkork
Copy link
Contributor

@tkork tkork commented Sep 16, 2024

No description provided.

@tkork tkork marked this pull request as ready for review September 17, 2024 08:56
@tkork
Copy link
Contributor Author

tkork commented Sep 17, 2024

Taitaa olla niin, että jatkopäätöksiltä pitää ottaa noi kentät pois

hakalap
hakalap reviewed Sep 18, 2024
View reviewed changes
backend/src/handler/email/AloituskuulutusHyvaksyntaEmailSender.ts Outdated
const aloituskuulutusRuotsiPDF = await this.getMandatoryProjektiFileAsAttachment(
aloituskuulutus.aloituskuulutusPDFt?.[Kieli.RUOTSI]?.aloituskuulutusPDFPath,
projekti,
`aloituskuulutusPDFPath SUOMI`
Copy link
Contributor

Choose a reason for hiding this comment

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

SUOMI -> RUOTSI

src/components/projekti/paatos/kuulutuksenTiedot/index.tsx
@@ -131,7 +131,7 @@ function KuulutuksenTiedotForm({ kirjaamoOsoitteet, paatosTyyppi, projekti }: Ku

Copy link
Contributor

Choose a reason for hiding this comment

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

Lisää kirjeTiedotettavillePDFPath: kirjeTiedotettavillePDF?.tiedosto || null, riville 94.

hakalap
hakalap reviewed Sep 18, 2024
View reviewed changes
backend/src/projekti/projektiHandler.ts Outdated
@@ -598,7 +598,7 @@ async function handleNahtavillaoloSaamePDF(dbProjekti: DBProjekti) {
dbProjekti.oid,
saamePDFt,
new ProjektiPaths(dbProjekti.oid).nahtavillaoloVaihe(nahtavillaoloVaihe),
["kuulutusPDF", "kuulutusIlmoitusPDF"],
["kuulutusPDF", "kuulutusIlmoitusPDF", "kirjeTiedotettavillePDF"],
[API.AsiakirjaTyyppi.NAHTAVILLAOLOKUULUTUS, API.AsiakirjaTyyppi.ILMOITUS_NAHTAVILLAOLOKUULUTUKSESTA_KUNNILLE_VIRANOMAISELLE],
Copy link
Contributor

Choose a reason for hiding this comment

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

Lisää asiakirjatyyppi kirjeelle.

hakalap
hakalap reviewed Sep 18, 2024
View reviewed changes
backend/src/projekti/projektiHandler.ts Outdated
@@ -598,7 +598,7 @@ async function handleNahtavillaoloSaamePDF(dbProjekti: DBProjekti) {
dbProjekti.oid,
saamePDFt,
new ProjektiPaths(dbProjekti.oid).nahtavillaoloVaihe(nahtavillaoloVaihe),
["kuulutusPDF", "kuulutusIlmoitusPDF"],
["kuulutusPDF", "kuulutusIlmoitusPDF", "kirjeTiedotettavillePDF"],
[API.AsiakirjaTyyppi.NAHTAVILLAOLOKUULUTUS, API.AsiakirjaTyyppi.ILMOITUS_NAHTAVILLAOLOKUULUTUKSESTA_KUNNILLE_VIRANOMAISELLE],
[API.Kieli.POHJOISSAAME, API.Kieli.POHJOISSAAME]
Copy link
Contributor

Choose a reason for hiding this comment

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

Lisää kolmas kieli.

hakalap
hakalap reviewed Sep 18, 2024
View reviewed changes
backend/src/projekti/projektiHandler.ts
@@ -619,7 +619,7 @@ async function handleHyvaksymisPaatosSaamePDF(dbProjekti: DBProjekti) {
dbProjekti.oid,
saamePDFt,
new ProjektiPaths(dbProjekti.oid).hyvaksymisPaatosVaihe(hyvaksymisPaatosVaihe),
["kuulutusPDF", "kuulutusIlmoitusPDF"],
["kuulutusPDF", "kuulutusIlmoitusPDF", "kirjeTiedotettavillePDF"],
[
API.AsiakirjaTyyppi.HYVAKSYMISPAATOSKUULUTUS,
API.AsiakirjaTyyppi.ILMOITUS_HYVAKSYMISPAATOSKUULUTUKSESTA_KUNNALLE_JA_TOISELLE_VIRANOMAISELLE,
Copy link
Contributor

Choose a reason for hiding this comment

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

Lisää asiakirjatyyppi ja kieli.

@hakalap
Copy link
Contributor

hakalap commented Sep 18, 2024

Vois vielä lisätä tähän PR:ään että vie pdf:n asianhallintaan. Suomi.fi:in menevään pdf:ään ton saamenkielisen pdf voinee lisätä eri PR:ssä.

@tkork tkork changed the title fix: HASSUYP-289 Lisää paikat saamenkielisille kiinteistönomistajien ja muistuttajien kirjeille nähtävilläoloon ja päätöksien kuuluttamiseen fix: HASSUYP-289 Lisää paikat saamenkielisille kiinteistönomistajien ja muistuttajien kirjeille nähtävilläoloon ja hyväksymispäätökseen Sep 19, 2024
@hakalap
Copy link
Contributor

hakalap commented Sep 19, 2024 •
edited
Loading

Nyt toi tiedostovalitisin antaa valita vain kuvia. Tallenna luonnos ei toimi nähtävilläolossa aina. Taitaa tuo filujen tallennus tapahtua rinnakkain tallennuksen kanssa.

@tkork
Copy link
Contributor Author

tkork commented Sep 19, 2024

Jäi näemmä accept proppi tuolt pois. Oli meinaan aiemmin kyllä SaameTiedostoValitisimessa se. Ihmettelen sitä fileiden tallennusta viel et mistä kiikastaa.

@tkork tkork merged commit 83fff83 into main Sep 19, 2024
1 check passed
@tkork tkork deleted the HASSUYP-289 branch September 19, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hakalap hakalap hakalap approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tkork @hakalap