francisguchie commented on pull request #1984: URL: https://github.com/apache/fineract/pull/1984#issuecomment-1034533445
> > 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... > > Comment has been addressed change the variableName from documentTypeValue to documentIssueCountry and not Id because we are saving countries name in String directly. @ptuomola Kindly review -- 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]
