francisguchie edited a comment on pull request #1984: URL: https://github.com/apache/fineract/pull/1984#issuecomment-1008603126
> 1. If the column documentTypeValue is meant to capture the issuing country for the document (which actually can make sense for all kinds of documents, not just passports), would it not make sense to call it that? So how about calling the column e.g. document_issue_country_id rather than document_type_value? Value seems to suggest it's the counterpart for the document_type_id, which is not the case... @ptuomola & @rrpawar96 , The reason why only passport is needed for this function is that, as is in (Apache fineract) the validation is such that the document id is supposed to be unique in the database irrespective of the document selected. For example, if user selects drivers license, and enters 123456 for client X then later selects passport and enters 123456 then a validation errors shows up. because 123456 has been used already. Come to reality- In Africa where i have worked most of my life, as countries are fighting to reduce the financial exclusion percentages **(some countries are at 80% financial exclusion)**, **new / stringent KYC rules are being put in place**. For example in East and West Africa the central banks are pushing for one document only and that is the National ID. All other IDs are not allowed as KYC documents. Meanwhile, for Foreigners, only the Passport is recognized when opening a "bank" account in any bank of the hosting country. This flexibility of various documents is being reduced to just National Id and Passports -- 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]
