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

feat: epäaktiivinen tila hyväksymispäätöskuulutuksen jälkeen #370

Merged
merged 1 commit into from
Sep 20, 2022
Merged

feat: epäaktiivinen tila hyväksymispäätöskuulutuksen jälkeen #370

merged 1 commit into from
Sep 20, 2022

Conversation

haapamakim
Copy link
Contributor

No description provided.

@haapamakim haapamakim requested review from tkork, kettunju and ValheKouneli and removed request for tkork and kettunju September 14, 2022 10:54
@ValheKouneli ValheKouneli self-assigned this Sep 16, 2022
ValheKouneli
ValheKouneli requested changes Sep 16, 2022
View reviewed changes
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.

Käännöksistä puuttuu käännös uudelle projektistatukselle "EPAAKTIIVINEN". Voisi miettiä, haluaako myös JATKOPAATOS_1:lle ja JATKOPAATOS_2:lle jo tässä vaiheessa käännöksen. Mutta ainakin EPAAKTIIVISELLE on hyvä laittaa, koska nyt on mahdollista saada projekti sellaiseen tilaan, ja se näkyy UI:ssa.

Olisko hyvä tehdä saman tien nappi sille, että projekti laitetaan epäaktiiviseen tilaan? Vähän niin kuin Juhani teki napin sille nähtävilläolovaiheen siirtämiselle menneisyyteen.

Virkamiespuolella näkyy nyt epäaktiiviselle projektille Hyväksymisvaihessa infoloota, jossa lukee "Kuulutus nähtäville asettamisesta on julkaistu 16.09.2022 13:43. Projekti näytetään kuulutuspäivästä lasketun määräajan jälkeen palvelun julkisella puolella suunnittelussa olevana. Kuulutusvaihe päättyy 01.01.2021." Kun projekti on epäaktiivinen (mutta ei vielä poistettu), siinä voisi lukea jotain muuta.

Muuten (ja tuota yhtä erillistä kommenttia) lukuun ottamatta hyvät muutokset. Tuo tilakäsittelyjuttu on näppärä ja hyvä muutos.

src/pages/api/test/[oid]/hyvaksymispaatosmenneisyyteen.dev.ts Show resolved Hide resolved
@haapamakim haapamakim force-pushed the feature/HASSU-964 branch 5 times, most recently from 36a067d to f462ba4 Compare September 20, 2022 08:28
ValheKouneli
ValheKouneli approved these changes Sep 20, 2022
View reviewed changes
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.

En bongannut päivitetyistä muutoksista sitä, että virkamiehelle näkyvä info olis muutettu, jos suunnitelma on epäaktiivinen (vai mikä se mun ehdotus olikaan), mut ihan sama :D Riittävän hyvä. Palataan siihen sitten jos tarvitsee.

@haapamakim haapamakim merged commit 30b2918 into main Sep 20, 2022
@haapamakim haapamakim deleted the feature/HASSU-964 branch September 20, 2022 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ValheKouneli ValheKouneli ValheKouneli approved these changes

@tkork tkork Awaiting requested review from tkork

@kettunju kettunju Awaiting requested review from kettunju

Assignees

@ValheKouneli ValheKouneli

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@haapamakim @ValheKouneli