Skip to content

Commit

Permalink
VA-436-2: Add oikaisuvaatimusosoitus liite to muutoshakemus paatos. I…
Browse files Browse the repository at this point in the history
…t should be there always 'ko. liite (3a) tulee mukaan aivan joka ikiseen päätökseen aina ja ikuisesti, joten siellä mitään poikkeuksia ei ole.'
  • Loading branch information
tulma95 committed Dec 3, 2024
1 parent 2b53a82 commit 2c0939e
Show file tree
Hide file tree
Showing 5 changed files with 51 additions and 4 deletions.
12 changes: 12 additions & 0 deletions playwright/tests/muutoshakemus/hakija-muutoshakemus-paatos.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -224,6 +224,12 @@ test('Hakija views muutoshakemus page', async ({ hakijaMuutoshakemusPaatosPage }
const currentValues = await hakijaMuutoshakemusPaatosPage.changedBudgetTableCells()
expect(sortedFormTable(currentValues)).toEqual(sortedFormTable(budgetExpectedItems))
})
await test.step('Oikaisuvaatimus liite is shown', async () => {
expect(hakijaMuutoshakemusPaatosPage.page.getByText('Liitteet')).toBeVisible()
expect(
hakijaMuutoshakemusPaatosPage.page.getByRole('link', { name: 'Oikaisuvaatimusosoitus' })
).toBeVisible()
})
})

export const svBudget: Budget = {
Expand Down Expand Up @@ -370,6 +376,12 @@ svTest(
]
expect(swedishBudgetRowDescriptions.sort()).toEqual(swedishBudgetRowNames.sort())
})
await svTest.step('Oikaisuvaatimus liite in Swedish', async () => {
await expect(hakijaMuutoshakemusPaatosPage.page.getByText('Bilagor')).toBeVisible()
await expect(
hakijaMuutoshakemusPaatosPage.page.getByRole('link', { name: 'Rättelseyrkande' })
).toBeVisible()
})

await svTest.step('päätös email', async () => {
const email = await parseMuutoshakemusPaatosFromEmails(hakemusID)
Expand Down
2 changes: 1 addition & 1 deletion server/resources/feature-flags.edn
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,5 @@
; https://jira.eduuni.fi/browse/VA-400

; https://jira.eduuni.fi/browse/VA-436
:muutospaatoksen-liite [:dev :test :aws-dev :aws-qa]
:muutospaatoksen-liite [:local :dev :test :aws-dev :aws-qa]
}
35 changes: 33 additions & 2 deletions soresu-form/web/va/MuutoshakemusPaatos.tsx
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
import React from 'react'
import React, { useEffect, useState } from 'react'
import moment from 'moment'

import { getProjectEndDate, getTalousarvio, isAcceptedWithChanges } from './Muutoshakemus'
import { TalousarvioTable } from './muutoshakemus/MuutosTaloudenKayttosuunnitelmaan'
import { useTranslations } from 'soresu-form/web/va/i18n/TranslationContext'
import { fiShortFormat } from 'soresu-form/web/va/i18n/dateformat'
import { fiShortFormat } from '../va/i18n/dateformat'
import {
Muutoshakemus,
Paatos,
Expand All @@ -16,6 +16,8 @@ import {
import './MuutoshakemusPaatos.less'
import { Role } from '../../../va-virkailija/web/va/types'
import { OsioPaatos, PaatosOsio } from './OsioPaatos'
import HttpUtil from '../HttpUtil'
import { EnvironmentApiResponse } from './types/environment'

type MuutoshakemusPaatosProps = Omit<PaatosState, 'paatos' | 'presenter'> & {
paatos: Omit<Paatos, 'id' | 'user-key' | 'updated-at'>
Expand Down Expand Up @@ -222,6 +224,21 @@ const LisatietojaSection: React.FC<{ presenter: Role | undefined }> = ({ present
)
}

function LiitteetSection() {
const { t, lang } = useTranslations()

const link = `liitteet/3a_oikaisuvaatimusosoitus_valtionavustuslaki_${lang}.pdf`
return (
<section className="muutoshakemus-paatos__section">
<div className="muutoshakemus-paatos__title">{t.muutoshakemus.paatos.liitteet}</div>

<div>
<a href={link}>{t.muutoshakemus.paatos.oikaisuvaatimusosoitus}</a>
</div>
</section>
)
}

export const MuutoshakemusPaatos = ({
hakemus,
muutoshakemus,
Expand All @@ -233,6 +250,19 @@ export const MuutoshakemusPaatos = ({
muutoshakemusUrl,
}: MuutoshakemusPaatosProps) => {
const { t } = useTranslations()
const [showLiitteet, setShowLiitteet] = useState(false)

useEffect(() => {
async function getMuutospaatoksenLiiteFlag() {
const data = await HttpUtil.get<EnvironmentApiResponse>('/environment')
// VA-436
if (data['feature-flags'].includes('muutospaatoksen-liite')) {
setShowLiitteet(true)
}
}
void getMuutospaatoksenLiiteFlag()
}, [])

const paattymispaiva = isAcceptedWithChanges(paatos['paatos-status-jatkoaika'])
? paatos.paattymispaiva
: muutoshakemus['haettu-kayttoajan-paattymispaiva']
Expand Down Expand Up @@ -283,6 +313,7 @@ export const MuutoshakemusPaatos = ({
presenter={presenter}
/>
<LisatietojaSection presenter={presenter} />
{showLiitteet && <LiitteetSection />}
</div>
)
}
4 changes: 4 additions & 0 deletions soresu-form/web/va/i18n/translations.ts
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,8 @@ export const translationsFi = {
lisatietoja: 'Lisätietoja',
phoneNumber: '029 533 1000 (vaihde)',
paatosDokumentti: 'Päätösdokumentti',
liitteet: 'Liitteet',
oikaisuvaatimusosoitus: 'Oikaisuvaatimusosoitus',
status: {
accepted: 'Opetushallitus on hyväksynyt haetut muutokset.',
rejected: 'Opetushallitus on hylännyt haetut muutokset.',
Expand Down Expand Up @@ -244,6 +246,8 @@ const translationsSv: Translations = {
lisatietoja: 'Mer information',
phoneNumber: '029 533 1000 (växel)',
paatosDokumentti: 'Beslutshandling',
oikaisuvaatimusosoitus: 'Rättelseyrkande',
liitteet: 'Bilagor',
status: {
accepted: 'Utbildningsstyrelsen har godkänt ändringarna enligt ändringsansökan.',
accepted_with_changes:
Expand Down
2 changes: 1 addition & 1 deletion soresu-form/web/va/types/environment.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ type LegacyFeatureFlag = { 'enabled?': boolean }
Add flags here
Currently empty string as no active flags
*/
export type FeatureFlag = ''
export type FeatureFlag = 'muutospaatoksen-liite'

export interface EnvironmentApiResponse {
name: string
Expand Down

0 comments on commit 2c0939e

Please sign in to comment.