Re: Merging Swagger API work for Fineract 1.2 release

2018-04-10 Thread Ed Cable
Aleks, So you were able to successfully get your build running with Sanyam's changes? Ed On Sat, Mar 31, 2018 at 2:23 AM, Aleksandar Vidakovic < chee...@monkeysintown.com> wrote: > Hi Sanyam, > > ... I should have read your email earlier ;-) ... just found out and fixed > the license check loca

Re: Merging Swagger API work for Fineract 1.2 release

2018-03-31 Thread Aleksandar Vidakovic
Hi Sanyam, ... I should have read your email earlier ;-) ... just found out and fixed the license check locally on my machine... I had a look at your changes... looks good to me. You need anything else? Cheers, Aleks On Fri, Mar 30, 2018 at 4:42 PM Sanyam Goel wrote: > Hi Avik, > > All the s

Re: Merging Swagger API work for Fineract 1.2 release

2018-03-30 Thread Sanyam Goel
Hi Avik, All the swagger files are removed from the License check as per Markus suggestion[ https://github.com/apache/fineract/pull/409#issuecomment-354417476 ], please see the link: https://github.com/apache/fineract/pull/409/files#diff-baadf2a612ff1f9e8d1a1347fe9b532cL113 And the branch is succ

Re: Merging Swagger API work for Fineract 1.2 release

2018-03-29 Thread Aleksandar Vidakovic
Hi Sanyam, I'll have a look at it and test it out. Cheers On Wed, Mar 28, 2018, 8:53 PM Avik Ganguly wrote: > Hi Sanyam, > > Last time (couple of months back) I tried to build your branch, it failed > to build and I remember having some discussion regarding the build issues > (although I can't

Re: Merging Swagger API work for Fineract 1.2 release

2018-03-28 Thread Avik Ganguly
Hi Sanyam, Last time (couple of months back) I tried to build your branch, it failed to build and I remember having some discussion regarding the build issues (although I can't seem to find that discussion in mail or Github). Are the build issues resolved or is there any manual steps still left w

Re: Merging Swagger API work for Fineract 1.2 release

2018-03-28 Thread Sanyam Goel
Hi Aleks, Have you got a chance to review the Pull Request? Please let me know if you want to discuss anything regarding the PR, we can schedule the meeting on google hangouts or skype and let's try to wrap up all the work and push it to the main codebase to boost up the release of Fineract 1.2.

Re: Merging Swagger API work for Fineract 1.2 release

2018-03-22 Thread Aleksandar Vidakovic
I'll try to get a look at this tomorrow On Thu, Mar 22, 2018, 10:04 PM Sanyam Goel wrote: > Hi Ed, > > All the GCI work was pushed on the same Pull Request #409. > Please let me know if I need to do any other changes in the PR. > > Thanks, > Sanyam > > On Fri, Mar 23, 2018 at 2:18 AM, Ed Cable

Re: Merging Swagger API work for Fineract 1.2 release

2018-03-22 Thread Sanyam Goel
Hi Ed, All the GCI work was pushed on the same Pull Request #409. Please let me know if I need to do any other changes in the PR. Thanks, Sanyam On Fri, Mar 23, 2018 at 2:18 AM, Ed Cable wrote: > Sanyam, > > I know that the Swagger work you did during GSOC for Mifos last summer is > all contai