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

OK-770 attachment deadline logic for kk application payments #1666

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

vaeinoe
Copy link
Contributor

@vaeinoe vaeinoe commented Dec 17, 2024

Even when the exemption question is answered, making the applicant exempt from application payment, the required attachment must still be present and valid by the attachment deadline.

If one or more of the mandatory attachments in the module is marked incomplete or missing, and the attachment deadline for the admission has already passed, set payment required for the person.

@vaeinoe vaeinoe marked this pull request as ready for review December 18, 2024 05:49
Comment on lines +578 to +579
:hakuajat [{:alkaa "2030-01-01T08:00:00",
:paattyy "2030-06-01T15:00:00"}]})
Copy link
Contributor Author

@vaeinoe vaeinoe Dec 18, 2024

Choose a reason for hiding this comment

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

Tätä käyttävät testit tulevat toimimaan ongelmitta seuraavat 5 vuotta: toivottavasti sillä välillä ataruun tehdään isompi remppa, eivätkä jälkipolvet joudu lisäämään tähän taas viittä tai kymmentä vuotta.

@@ -23,6 +23,23 @@
(def kk-application-payment-document-exempt-options
(dissoc kk-application-payment-document-options :no-document-option-value))

(def kk-application-payment-exempt-attachment-keys
Copy link
Contributor Author

@vaeinoe vaeinoe Dec 18, 2024

Choose a reason for hiding this comment

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

Tästä puuttuu siis modulin liitteistä none-passport-attachment tarkoituksella, koska se ei kuulu vapauttavaan kysymykseen.

@vaeinoe vaeinoe force-pushed the OK-770-kk-payment-attachment-deadline-logic branch from ce2ba3f to 5e0de7d Compare December 18, 2024 08:46
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.

2 participants