feat(kesaseteli,admin): admin employer applications and vouchers#3840
Merged
nikomakela merged 4 commits intomainfrom Jan 21, 2026
Merged
feat(kesaseteli,admin): admin employer applications and vouchers#3840nikomakela merged 4 commits intomainfrom
nikomakela merged 4 commits intomainfrom
Conversation
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
b5c10c0 to
1860f98
Compare
| ] | ||
|
|
||
| def has_add_permission(self, request): | ||
| """Disable adding new applications.""" |
Collaborator
There was a problem hiding this comment.
e.g.
Suggested change
| """Disable adding new applications.""" | |
| """Disable adding new employer summer vouchers.""" |
| return False | ||
|
|
||
| def has_delete_permission(self, request, obj=None): | ||
| """Disable deleting applications.""" |
Collaborator
There was a problem hiding this comment.
e.g.
Suggested change
| """Disable deleting applications.""" | |
| """Disable deleting employer summer vouchers.""" |
Collaborator
karisal-anders
left a comment
karisal-anders
left a comment
There was a problem hiding this comment.
Changing the verbose names for EmployerApplication and EmployerSummerVoucher models -> Getting less ambiguous names for them in Django admin would be good. Otherwise looks quite ok already, let's have a look when this is not a draft & tests have passed (Probably tests are only run for "merge to main" PRs?)
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
2 times, most recently
from
eb1658e to
a1b0d9c
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
1860f98 to
388697b
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
a1b0d9c to
c87c52e
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
388697b to
9efe24b
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
c87c52e to
fd309ce
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
9efe24b to
1261315
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
fd309ce to
f2e0ce5
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
1261315 to
df557a2
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
f2e0ce5 to
f166d71
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
df557a2 to
fde5778
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
f166d71 to
1806d31
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
fde5778 to
a632676
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
1806d31 to
37a0b53
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
a632676 to
19140ca
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
3e828d6 to
a3fd320
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
19140ca to
a5256b9
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
a3fd320 to
d65a340
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
a5256b9 to
ddb4e55
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
d65a340 to
46aa72b
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
3c5e8c4 to
b83a336
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
d2cd64b to
4af556a
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
b83a336 to
6c93fcf
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
4af556a to
fedb3ee
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
6c93fcf to
e40e7ef
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
fedb3ee to
fa60765
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
e40e7ef to
6424f60
Compare
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
fa60765 to
8285953
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
6424f60 to
04a0d76
Compare
karisal-anders
approved these changes
Jan 21, 2026
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
8285953 to
4c67ca7
Compare
nikomakela
force-pushed
the
YJDH-791-youth-application-admin
branch
from
04a0d76 to
e9f6790
Compare
YJDH-791. Django admin site can be used to list and view organisations and their data. No additions, changes or deletions are allowed via admin site. NOTE: Registering companies to admin site makes it possible to use company as a search field in other admin views.
YJDH-791 YJDH-792. Register `EmployerApplication` and `EmployerSummerVoucher` with multiple filters and search fields to Django's admin site. A `EmployerApplication` can only be updated from it's employment details and employee info parts, it cannot be deleted and a new one cannot be created via admin site.
YJDH-791. Change verbose names for employer application and employer summer voucher, because without context it was unclear what voucher or application was meant. This was especially frustrating when using admin site, where the models are listed in left side panel.
nikomakela
force-pushed
the
YJDH-791-employer-application-admin
branch
from
4c67ca7 to
e456f1d
Compare
|
YJDH-KESASETELI-API branch is deployed to platta: https://yjdh-kesaseteli-pr3840.api.dev.hel.ninja 🚀🚀🚀 |
|
|
YOUTH branch is deployed to platta: https://nuortenkesaseteli-pr3840.dev.hel.ninja 🚀🚀🚀 |
|
EMPLOYER branch is deployed to platta: https://kesaseteli-pr3840.dev.hel.ninja 🚀🚀🚀 |
|
HANDLER branch is deployed to platta: https://kesaseteli-handler-ui-pr3840.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://nuortenkesaseteli-pr3840.dev.hel.ninja 😆🎉🎉🎉 |
TestCafe result is success for https://kesaseteli-handler-ui-pr3840.dev.hel.ninja 😆🎉🎉🎉 |
TestCafe result is success for https://kesaseteli-pr3840.dev.hel.ninja 😆🎉🎉🎉 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
YJDH-791 YJDH-792.
Register
Organisation,EmployerApplicationandEmployerSummerVoucherwith multiple filters and search fields to Django's admin site.A
EmployerApplicationcan only be updated from it's employment details and employee info parts, it cannot be deleted and a new one cannot be created via admin site.Applications:
Summer vouchers: