danishjamal104 commented on pull request #1810:
URL: https://github.com/apache/fineract/pull/1810#issuecomment-907805720


   > LGTM... I think there may be some other possible fields in 
GetClientsGroups but this is definitely an improvement over the previous 
version.
   > 
   > Though it would be great to get integration tests for all of these 
included in the build :-)
   
   Yes, since I am working with the SDK integration in one of the MIFOS's 
client application, I am encountering more and more errors in swagger 
resources, so  Ill keep fixing them in future with the proper test 
implementation also.


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