[GitHub] [fineract] thesmallstar commented on pull request #1330: Interop KYC and Interop Loan Disbursement
thesmallstar commented on pull request #1330: URL: https://github.com/apache/fineract/pull/1330#issuecomment-696935962 @SubhamPramanik thank you for the PR, I see a checkstyle error in your code, that means address_line_1 that you made does not match the required regex it should. I also see a AvoidHidingCauseException, that happens when you catch a exception and throw another exception essentially hiding what actually caused the exception! Will you fix this? cc @vorburger The checkstyles are helping!! 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [fineract] thesmallstar commented on pull request #1330: Interop KYC and Interop Loan Disbursement
thesmallstar commented on pull request #1330: URL: https://github.com/apache/fineract/pull/1330#issuecomment-696935962 @SubhamPramanik thank you for the PR, I see a checkstyle error in your code, that means address_line_1 that you made does not match the required regex it should. I also see a AvoidHidingCauseException, that happens when you catch a exception and throw another exception essentially hiding what actually caused the exception! Will you fix this? cc @vorburger The checkstyles are helping!! 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org