subhranshudash13-dotcom commented on issue #534: URL: https://github.com/apache/fineract-backoffice-ui/issues/534#issuecomment-5586306962
Hey @Aman-Mittal, I looked into this and found that `clients-list.component.ts` is passing `1` as the `legalForm` to `getClients()`, which maps to `person`. This causes the main client list to exclude entity clients, while Advanced Search doesn't have this restriction. My fix : I am thinking of removing the hardcoded `legalForm` so that both person and entity clients are included in the main list. Before I proceed, could you confirm the expected behavior here? Should the list simply show "both person and entity clients by default", with no legal-form filtering, or is the intention to "extend the UI with a Legal Form filter" (Person / Entity / All) so users can choose what they want to see? Once I know the expected behavior, I'll proceed with the appropriate change. -- 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]
