Skip to content

Comments

fix(benefit): application disappears from "Maksuerät" tab#3861

Merged
ext-hjasplund merged 1 commit intomainfrom
hl-1719-application-disappears
Feb 16, 2026
Merged

fix(benefit): application disappears from "Maksuerät" tab#3861
ext-hjasplund merged 1 commit intomainfrom
hl-1719-application-disappears

Conversation

@ext-hjasplund
Copy link
Collaborator

@ext-hjasplund ext-hjasplund commented Feb 2, 2026

Description

Fixed query to allow for applications to remain on "Maksuerät" tab.

Related Issue(s)

HL-1719

Motivation and Context

Applications disappeared from "Maksuerät" tab after the first installment.

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@ext-hjasplund ext-hjasplund marked this pull request as ready for review February 2, 2026 07:27
@ext-hjasplund ext-hjasplund requested a review from a team as a code owner February 2, 2026 07:27
@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

Copy link
Contributor

@mhieta mhieta left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@mhieta mhieta self-requested a review February 4, 2026 13:08
Copy link
Contributor

@mhieta mhieta left a comment

Choose a reason for hiding this comment

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

Actually this fix does not seem to change anything, because the should_filter_archived-variable is always True in the first if and always False in the second else. Or?

Also better add explicit tests for the fix to verify that the fix does what it is supposed to do.

@JJ-gitti
Copy link

JJ-gitti commented Feb 4, 2026

The real issue seems to be in this function that filters out appliactions which have due_date in past.
The due_date can be in past if there something to clarify for the 2. instalment.

def _get_application_pks_with_instalments(self) -> List[int]: return Application.objects.filter( status=ApplicationStatus.ACCEPTED, calculation__instalments__due_date__gte=timezone.now().date(), calculation__instalments__instalment_number=2, calculation__instalments__status__in=[ InstalmentStatus.ACCEPTED, InstalmentStatus.ERROR_IN_TALPA, InstalmentStatus.WAITING, ], )

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@ext-hjasplund ext-hjasplund force-pushed the hl-1719-application-disappears branch from d510f80 to 6136e50 Compare February 6, 2026 12:42
@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@ext-hjasplund ext-hjasplund force-pushed the hl-1719-application-disappears branch from 2d5e2ea to 2ad820b Compare February 13, 2026 10:03
@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@ext-hjasplund ext-hjasplund force-pushed the hl-1719-application-disappears branch from 2ad820b to be0d82a Compare February 16, 2026 10:02
@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

Fix the issue where the applications disappear
from the "Maksuerät" tab.
Created a test.

Refs: HL-1719
@azure-pipelines
Copy link

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

@ext-hjasplund ext-hjasplund force-pushed the hl-1719-application-disappears branch from be0d82a to c3986dd Compare February 16, 2026 10:39
@sonarqubecloud
Copy link

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@charn charn requested a review from mhieta February 16, 2026 11:24
Copy link
Contributor

@charn charn left a comment

Choose a reason for hiding this comment

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

LGTM! 👍 :shipit:

@ext-hjasplund ext-hjasplund merged commit cd790cd into main Feb 16, 2026
66 checks passed
@ext-hjasplund ext-hjasplund deleted the hl-1719-application-disappears branch February 16, 2026 11:36
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.

4 participants