Samer-Melhem-FOO commented on PR #6188: URL: https://github.com/apache/fineract/pull/6188#issuecomment-5178211540
<img width="907" height="430" alt="Screenshot 2026-08-04 141147" src="https://github.com/user-attachments/assets/8aebb835-0e8e-44e7-88df-a218837f5755"; /> Looking at the screenshot of the current UI (toggle off, "Show Closed Accounts"), all clients — including closed ones — are already returned by default. That doesn't quite line up with the generic status-filtering solution being proposed in this PR. What we'd suggest instead: keep the generic status filtering, but flip the UI semantics — rename the toggle label from "Show Closed Accounts" to "Hide Closed Accounts". In its default (off) state, all clients continue to be returned as today, including closed ones. When toggled on, we hide closed accounts by adjusting the status filter param accordingly. This keeps default behavior unchanged, gives users an explicit way to exclude closed accounts, and removes the mismatch between the switch's current label/intent and the filtering approach proposed here. Posting this as pending your feedback — want to align on this before we land on a fully working solution. @adamsaghy @IOhacker -- 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]
