fineract-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [fineract] ptuomola edited a comment on pull request #914: FINERACT-723: Fixing date comparisons
Date Thu, 21 May 2020 03:14:23 GMT

ptuomola edited a comment on pull request #914:
URL: https://github.com/apache/fineract/pull/914#issuecomment-631853470


   > See #627 do we need to remove the hard coded Timezone we are setting in Travis now
with this PR? @vorburger ..Maybe?
   
   The Utils class used by RestAssured also has the tenant timezone hardcoded:
   
   in org.apache.fineract.integrationtests.common.Utils:
       public static final String TENANT_TIME_ZONE = "Asia/Kolkata";
   
   A better solution would of course be to change the Utils class to look this up from the
database.
   
   After my PR, the current IT scope seems to work as everything is hardcoded to point to
the same timezone (Asia/Kolkata). 
   
   To make everything work with any timezone for tenant/database/application server without
any hardcoding, then we will need to review/update all code and queries using either application
server date and database date.
   
   


----------------------------------------------------------------
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