-
Notifications
You must be signed in to change notification settings - Fork 10
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
Hl 963 ps #2281
Conversation
API branch is deployed to platta: https://helsinkilisa-pr2281.api.dev.hel.ninja/healthz 🚀🚀🚀 |
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2281.dev.hel.ninja 🚀🚀🚀 |
APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2281.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr2281.dev.hel.ninja 😆🎉🎉🎉 |
API branch is deployed to platta: https://helsinkilisa-pr2281.api.dev.hel.ninja/healthz 🚀🚀🚀 |
APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2281.dev.hel.ninja 🚀🚀🚀 |
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2281.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr2281.dev.hel.ninja 😆🎉🎉🎉 |
[yjdh] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
[yjdh] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
[yjdh] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
[yjdh] Kudos, SonarCloud Quality Gate passed! |
API branch is deployed to platta: https://helsinkilisa-pr2281.api.dev.hel.ninja/healthz 🚀🚀🚀 |
HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2281.dev.hel.ninja 🚀🚀🚀 |
APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2281.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://helsinkilisa-ui-pr2281.dev.hel.ninja 😆🎉🎉🎉 |
Description ✨
Submitting an application manually as a handler does not create rows into the
bf_calculation_paysubsidy
table in the database, because the JSON payload it submits has an emptypay_subsidies
list evaluates to notNone
which causes a branch in the calculation logic to skip the database insert. This PR fixes the issue by checking if the application is submitted by a handler and then makes sure thatpay_subsidy_data
isNone
andpay_subsidies
is is not present in thevalidated_data
dict.Issues 🐛
Testing ⚗️
Pytest: pytest applications/tests/test_handler_application.py
Manually: create a new application as a handler and then as a handler process the application.
Screenshots 📸
Additional notes 🗒️