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 2021/08/20 12:39:11 UTC

[GitHub] [fineract] ptuomola commented on pull request #1820: field name issue fixed FINERACT-1382

ptuomola commented on pull request #1820:
URL: https://github.com/apache/fineract/pull/1820#issuecomment-902661907


   True - there is no risk. But what I was hoping for is that we could create tests for these at the same time...
   
   > On 20 Aug 2021, at 8:37 PM, Awasum Yannick ***@***.***> wrote:
   > 
   > 
   > @awasum approved this pull request.
   > 
   > LGTM. Even though its difficult to test every Swagger API change. This is not risky to merge.
   > 
   > —
   > You are receiving this because you are subscribed to this thread.
   > Reply to this email directly, view it on GitHub <https://github.com/apache/fineract/pull/1820#pullrequestreview-734983241>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AASJVCXZAGJ72Q4HMIMWME3T5ZD7VANCNFSM5CE57WXQ>.
   > Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email>.
   > 
   
   


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