Skip to content

Comments

feat(benefit): applicant ui to reflect 2026 changes#3845

Closed
ext-hjasplund wants to merge 6 commits intomainfrom
hl-1706-new-ui-2026
Closed

feat(benefit): applicant ui to reflect 2026 changes#3845
ext-hjasplund wants to merge 6 commits intomainfrom
hl-1706-new-ui-2026

Conversation

@ext-hjasplund
Copy link
Collaborator

@ext-hjasplund ext-hjasplund commented Jan 21, 2026

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

Changed UI fields according to new rules.
Changed validation logic in the backend.
Fixed browser and backend tests.

Refs: HL-1706
@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😆🎉🎉🎉

@ext-hjasplund ext-hjasplund marked this pull request as ready for review January 21, 2026 06:02
@ext-hjasplund ext-hjasplund requested review from a team as code owners January 21, 2026 06:02
@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

Copy link
Contributor

@voneiden voneiden left a comment

Choose a reason for hiding this comment

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

BE puolen validointi kaipaisi mielestäni vielä päivityksiä:

  1. API sallii tällä hetkellä, että hakemus lähetetään ilman apprenticeship_program kenttää tai null arvolla. FE:n puolella kenttä on kuitenkin laitettu pakolliseksi. Olettaisin, että tulisi olla BE:ssä myös nyt pakollinen?

  2. pay_subsidy_granted on kovakoodattu frontissa lähtemään arvolla NOT_GRANTED, mutta BE:ssä arvoa ei ole rajoitettu.

  3. 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)

Copy link

@timwessman timwessman left a comment

Choose a reason for hiding this comment

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

Looks good on FE side!

@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

Changed apprenticeship validation.
Fixed tests.

Refs: HL-1706
@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥

Changed apprenticeship validation.
Fixed tests.

Refs: HL-1706
@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥

Changed apprenticeship validation.
Fixed tests.

Refs: HL-1706
@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3845.api.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 🚀🚀🚀

@azure-pipelines
Copy link

TestCafe result is success for https://helsinkilisa-ui-pr3845.dev.hel.ninja 😆🎉🎉🎉

@azure-pipelines
Copy link

TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥

1 similar comment
@azure-pipelines
Copy link

TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3845.dev.hel.ninja 😿💢💥💥

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