feat(benefit): applicant ui to reflect 2026 changes#3845
feat(benefit): applicant ui to reflect 2026 changes#3845ext-hjasplund wants to merge 6 commits intomainfrom
Conversation
Changed UI fields according to new rules. Changed validation logic in the backend. Fixed browser and backend tests. Refs: HL-1706
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
TestCafe result is success for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
voneiden
left a comment
There was a problem hiding this comment.
BE puolen validointi kaipaisi mielestäni vielä päivityksiä:
-
API sallii tällä hetkellä, että hakemus lähetetään ilman
apprenticeship_programkenttää tainullarvolla. FE:n puolella kenttä on kuitenkin laitettu pakolliseksi. Olettaisin, että tulisi olla BE:ssä myös nyt pakollinen? -
pay_subsidy_grantedon kovakoodattu frontissa lähtemään arvollaNOT_GRANTED, mutta BE:ssä arvoa ei ole rajoitettu. -
https://github.com/City-of-Helsinki/yjdh/pull/3845/files#diff-e2eff806ffa41db0f4ff4a1b85b0b0e1d6c6f8750640f09f4a769fd9adb8b912L997-L1009 Tällä ei varmaan pitäisi olla enää mitään virkaa (olettaen, että sallitaan vain NOT_GRANTED arvot serializerin kautta)
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
Changed apprenticeship validation. Fixed tests. Refs: HL-1706
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥 |
Changed apprenticeship validation. Fixed tests. Refs: HL-1706
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥 |
Changed apprenticeship validation. Fixed tests. Refs: HL-1706
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
Fixed tests. Refs: HL-1706
Changed validation. Refs: HL-1706
|
YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀 |
|
APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉 |
TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥 |
1 similar comment
TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥 |
Description
Changed UI fields according to new rules.
Changed localization.
Changed validation logic in the backend.
Fixed browser and backend tests.
Related Issue(s)
HL-1706
Motivation and Context
Rules for benefit application changed in the beginning of 2026.
How Has This Been Tested?
Tests existed beforehand