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.

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

Lisätään sijoitusehdotuksen hylkäyksen syy hakemuksen muistiinpanoihin #6000

Merged
merged 5 commits into from
Nov 20, 2024

Conversation

patari
Copy link
Collaborator

@patari patari commented Nov 19, 2024

Kun johtaja hylkää sijoitusehdotuksen ja kirjaa hylkäykseen syyn, teksti katoaa, kun palveluohjaus palauttaa sijoitusehdotuksen sieltä johtajan tarkistettavana tilasta taaksepäin.

Tämän muutoksen myötä palveluohjaus näkee hylkäyksen syyn hakemuksen muistiinpanoista.

@patari patari added the enhancement Uusi toiminnallisuus tai parannus label Nov 19, 2024
@@ -786,6 +793,10 @@ data class PlacementProposalConfirmationUpdate(
val otherReason: String?,
)

data class AcceptPlacementProposalRequest(
val rejectReasons: Map<PlacementPlanRejectReason, String>
Copy link
Contributor

Choose a reason for hiding this comment

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

Tämän voisi nimetä uudelleen:

Suggested change
val rejectReasons: Map<PlacementPlanRejectReason, String>
val rejectReasonTranslations: Map<PlacementPlanRejectReason, String>

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

👍

.forEach {
val reason =
when (it.unitRejectReason) {
PlacementPlanRejectReason.OTHER -> it.unitRejectOtherReason
Copy link
Contributor

Choose a reason for hiding this comment

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

Pitäiskö tässä prefixata muistiinpanon sisältöä hieman, jotta se on yksiselitteisempi:

Niin, että muistiinpanon sisällöksi tulee esim.
"Sijoitusehdotus hylätty - Päiväkoti täynnä" (Tampere; REASON_1)
"Sijoitusehdotus hylätty - Sisäilma tai muu rakenteellinen syy" (Tampere; REASON_2)
"Sijoitusehdotus hylätty - Muu syy: Emme hyväksy maanantaisin lähetettyjä hakemuksia lainkaan" (OTHER)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

👍

@patari patari merged commit c3def96 into master Nov 20, 2024
29 checks passed
@patari patari deleted the placement-plan-reject-reason-copy branch November 20, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Uusi toiminnallisuus tai parannus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants