-
Notifications
You must be signed in to change notification settings - Fork 0
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: HASSUYP-354 & HASSUYP-318 Aineistokategorisoinnin päivitys rata- ja yleissuunnitelmat #1243
Conversation
…lemassa olevissa aineistoissa
@@ -41,11 +46,7 @@ export default function LausuntopyyntoAineistoPage(props: Props): ReactElement { | |||
</Notification> | |||
)} | |||
{aineistot?.length && <H2>Suunnitelma</H2>} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vaikka ei kuulu tähän PR:ään niin tän vois muuttaa {aineistot?.length ? <H2>Suunnitelma</H2> : ""}
muotoon kun tulostaa 0 sivulle, jos ei aineistoja.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tää olikin korjattu toisessa PR:ssä.
Yksi kategoria haluttiin vain yleissuunnitelmiin (ys_osa_c_siltasuunnitelmat_ja_muut_taitorakenteet), mikä ajoi muokkaamaan toteutusta siihen suuntaan, että kategorisoinnit on tyyppikohtaisia.
Lisäksi yksi kategoria toivottiin poistettavaksi (ulkopuoliset_rakenteet), mutta kategorioita ei voida poistaa sillä ne voivat olla olemassa olevissa julkaisuissa. Kategoria voidaan asettaa deprekoiduksi, jolloin kyseistä kategoriaa ei enää näytetä aineistomuokkauslomakkeilla.
Lisäksi päivitetty hakulauseet Confluence-dokumentiaation tasalle.