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