Looks good. I'd only like to ask for one change before I merge: Please give the notification user and the password constants more specific names. e.g. USER_IDENTIFIER -> NOTIFICATION_USER_IDENTIFIER
Once you improve the application authentication code for accessing customer, you'll probably want to update this code again. But let's get this bit merged now anyways. [ Full content available at: https://github.com/apache/fineract-cn-demo-server/pull/20 ] This message was relayed via gitbox.apache.org for [email protected]
