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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
