vidakovic commented on code in PR #5811:
URL: https://github.com/apache/fineract/pull/5811#discussion_r3310135596


##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/account/api/AccountTransfersApiResource.java:
##########
@@ -56,9 +61,9 @@
 public class AccountTransfersApiResource {
 
     private final PlatformSecurityContext context;
-    private final DefaultToApiJsonSerializer<AccountTransferData> 
toApiJsonSerializer;
-    private final PortfolioCommandSourceWritePlatformService 
commandsSourceWritePlatformService;
     private final AccountTransfersReadPlatformService 
accountTransfersReadPlatformService;
+    private final PortfolioCommandSourceWritePlatformService 
commandsSourceWritePlatformService;

Review Comment:
   Nope, we want to remove these legacy command processing components. Only 
read services and the new `CommandDispatcher` can be in the JAX-RS resource 
classes.



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