fineract-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [fineract] ptuomola commented on pull request #1179: FINERACT-826 Migrate to java.time from Joda API
Date Thu, 16 Jul 2020 10:05:19 GMT

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:
users@infra.apache.org



Mime
View raw message