Skip to content

Support setting approval and validity start date when copying a plan#567

Merged
LKajan merged 3 commits intomainfrom
mahdollista-kaavan-asettaminen-voimaan
Jan 30, 2026
Merged

Support setting approval and validity start date when copying a plan#567
LKajan merged 3 commits intomainfrom
mahdollista-kaavan-asettaminen-voimaan

Conversation

@Mtk112
Copy link
Contributor

@Mtk112 Mtk112 commented Jan 14, 2026

Muokattu copy_plan lambdaa, niin että sille voi syöttää period_of_validity_start ja period_of_validity_start_end päivämäärät.

Puuttuu mm. testien päivitys

@Mtk112
Copy link
Contributor Author

Mtk112 commented Jan 14, 2026

@LKajan @nmaarnio

@LKajan LKajan force-pushed the mahdollista-kaavan-asettaminen-voimaan branch from fa70b47 to 123341a Compare January 27, 2026 06:51
@LKajan LKajan changed the title Refaktoroitu copy_plan lambda Support setting approval and validity start date when copying a plan Jan 27, 2026
@LKajan LKajan marked this pull request as ready for review January 27, 2026 08:00
@LKajan LKajan requested a review from nmaarnio January 27, 2026 08:00
Copy link
Contributor

@nmaarnio nmaarnio left a comment

Choose a reason for hiding this comment

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

Nyt taitaa mennä niin, että jos käyttäjä koittaa ohittaa joko "Hyväksytty kaava" tai "Vomassa" elinkaaren vaiheet, kopiointi ei onnistu, koska päivämäärät vaaditaan (eikä niitä voi syöttää käyttöliittymässä kuin mainituille elinkaaren vaiheille). Mutta elinkaaren vaiheita ei varmaan kuulukaan ohittaa

@LKajan
Copy link
Collaborator

LKajan commented Jan 28, 2026

Nyt taitaa mennä niin, että jos käyttäjä koittaa ohittaa joko "Hyväksytty kaava" tai "Vomassa" elinkaaren vaiheet, kopiointi ei onnistu, koska päivämäärät vaaditaan (eikä niitä voi syöttää käyttöliittymässä kuin mainituille elinkaaren vaiheille). Mutta elinkaaren vaiheita ei varmaan kuulukaan ohittaa

Ymmärtääkseni sekä hyväksytty, että voimassa -vaiheet ovat molemmat pakollisia vaiheita, jotka tulee toimittaa ryhtiin. Näitä ei siis voi ohittaa.

@LKajan LKajan force-pushed the mahdollista-kaavan-asettaminen-voimaan branch from bc8b3dd to f8ca111 Compare January 28, 2026 08:48
@LKajan LKajan force-pushed the mahdollista-kaavan-asettaminen-voimaan branch from f8ca111 to ecd63f3 Compare January 28, 2026 13:27
@LKajan LKajan merged commit d7ab5a8 into main Jan 30, 2026
2 checks passed
@LKajan LKajan deleted the mahdollista-kaavan-asettaminen-voimaan branch January 30, 2026 07:34
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.

3 participants