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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
