ptuomola commented on pull request #1179:
URL: https://github.com/apache/fineract/pull/1179#issuecomment-659311773


   Hi - in all the places where this currently calls ZoneId.systemDefault(), 
should we not be retrieving the appropriate tenant timezone instead? I know 
that's not what the previous code does, but wouldn't that be the right 
behaviour for any logic that is related to a specific tenant - i.e. we should 
be using the tenant's timezone for any dates? 


----------------------------------------------------------------
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:
[email protected]


Reply via email to