You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@fineract.apache.org by "Samer-Melhem-FOO (via GitHub)" <gi...@apache.org> on 2026/09/30 08:31:25 UTC

Re: [PR] FINERACT-2714: Add a client status filter to the client text search [fineract]

Samer-Melhem-FOO commented on PR #6188:
URL: https://github.com/apache/fineract/pull/6188#issuecomment-5907340966

   @adamsaghy @IOhacker Reworked as you suggested: the `excludeClosed` flag is replaced by a generic status filter. `ClientTextSearch` now takes an optional `status` list, e.g. `"status": ["ACTIVE", "PENDING"]`, and only clients in one of those statuses are returned. No filter (omitted or empty) returns clients in every status, so the default behaviour is unchanged and the UI can keep fetching closed clients only as an extra step.
   
   Integration tests cover no filter, a single status, `CLOSED` only, and several statuses; the whole `ClientSearchTest` passes locally. Rebased onto the latest develop (the conflicts are resolved) and squashed into one commit. I've also updated the PR title and description.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@fineract.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org