adamsaghy commented on code in PR #4364:
URL: https://github.com/apache/fineract/pull/4364#discussion_r1967623192


##########
fineract-provider/src/main/java/org/apache/fineract/infrastructure/event/external/service/serialization/serializer/loan/LoanBusinessEventSerializer.java:
##########
@@ -56,6 +59,7 @@ public class LoanBusinessEventSerializer implements 
BusinessEventSerializer {
     private final LoanSummaryBalancesRepository loanSummaryBalancesRepository;
     @Lazy
     private final LoanSummaryProviderDelegate loanSummaryProviderDelegate;
+    private final CustomDataSerializer customDataSerializer;

Review Comment:
   I would have it as a list, and if there is any implementation we are calling 
each serializer one by one and adding their piece of code into the `customData` 
field. What do you think?



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

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to