ptuomola commented on PR #2000: URL: https://github.com/apache/fineract/pull/2000#issuecomment-1107438619
The two things I think would be good to address / test are: 1. Backward compatibility: I don't think we want to add a new mandatory parameter to every single API signature. It would be better if the parameter was optional, and everything would work as-is if the parameter is not passed in 2. Optionality: I don't think everyone wants to set up and manage a new Redis database just to support this feature. So it should be possible to switch this on / off depending on whether this feature is required or not -- 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]
