You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@fineract.apache.org by GitBox <gi...@apache.org> on 2022/01/03 03:58:08 UTC

[GitHub] [fineract] vidakovic commented on pull request #2013: Upgrade to JDK 17

vidakovic commented on pull request #2013:
URL: https://github.com/apache/fineract/pull/2013#issuecomment-1003855638


   @ptuomola ... looks like a lot of changes, but in the end it boils down to one missing GSON JsonSerializer (added `org/apache/fineract/infrastructure/core/api/LocalTimeAdapter.java`) that is needed to make things work with JDK17. There is a new static method `GoogleGsonSerializerHelper.createGsonBuilder()` that should be used to instantiate `GsonBuilder` instead of using `new GsonBuilder()` ... well, the method was there before, but not static and unused... I guess it was just forgotten.


-- 
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: commits-unsubscribe@fineract.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org