Sure Santosh. I guess I will have to update my version of Fineract 1.0. Any time after the 27th is fine with me.
Thanks, Courage. On Fri, Dec 21, 2018 at 12:12 PM Santosh Math < [email protected]> wrote: > Hi Courage, > Good to know the PRs are merged. I know you're busy, but Can we have a call > so that you can help with testing the scenarios? > > On Fri, Dec 21, 2018 at 4:03 PM Avik Ganguly <[email protected]> > wrote: > > > Hi Courage / Ed, > > > > Yes, that PR was merged with some minor changes due to conflicts after > > rebasing your code. The commit history should include all your changes. > > > > There was another open PR which looked like a subset of PR #424 so I > closed > > that as well. > > > > Is there a subsequent PR which needs to be reviewed which we are talking > > about? > > > > Regards, > > Avik. > > ᐧ > > > > On Thu, Dec 20, 2018 at 9:14 PM Courage Angeh <[email protected]> > > wrote: > > > > > Hi Ed, > > > > > > The PR was merged by Avik from this PR: > > > https://github.com/apache/fineract/pull/424 > > > > > > Avik please can you confirm that? > > > > > > Thanks, > > > Courage > > > > > > On Thu, Dec 20, 2018 at 8:52 AM Ed Cable <[email protected]> wrote: > > > > > > > Courage, > > > > > > > > It looks like that pull request wasn't actually merged. > > > > > > > > Avik, it was closed by asfgit on Dec 13, 2017 with comments from your > > > end. > > > > Can you shine light on what changes Courage needs to make for PR to > be > > > > merged? > > > > > > > > Thanks, > > > > > > > > Ed > > > > > > > > On Tue, Dec 18, 2018 at 7:09 AM Courage Angeh < > [email protected]> > > > > wrote: > > > > > > > > > Hi Ed, > > > > > > > > > > Sorry for the late reply. Here is the link to the documentation > used > > to > > > > > configure notifications for an event on Fineract. > > > > > > > > > > > > > > > > > > > > https://mifosforge.jira.com/wiki/spaces/docs/pages/153387033/Configure+Notifications > > > > > > > > > > Thanks, > > > > > Courage. > > > > > > > > > > On Tue, Dec 11, 2018 at 1:46 PM Ed Cable <[email protected]> > wrote: > > > > > > > > > > > Thanks for the update. So we can close out the ticket on JIRA? > Can > > > you > > > > > > point us to the notifications framework documentation again? For > > some > > > > of > > > > > > the ongoing work with mobile apps in the community, understanding > > the > > > > > > points of integration is necessary. > > > > > > > > > > > > Ed > > > > > > > > > > > > On Tue, Dec 11, 2018 at 9:53 AM Courage Angeh < > > > [email protected]> > > > > > > wrote: > > > > > > > > > > > > > Hi Ed, > > > > > > > > > > > > > > Hope this email finds you well. > > > > > > > My code was merged. Here is a link to one of the new files I > > > created > > > > > > during > > > > > > > GSCOC'17: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/fineract/blob/develop/fineract-provider/src/main/java/org/apache/fineract/notification/data/TopicData.java > > > > > > > > > > > > > > I will set-up my system to test the notification functionality > > > again. > > > > > > > > > > > > > > Thanks, > > > > > > > Courage. > > > > > > > > > > > > > > On Tue, Dec 11, 2018 at 8:25 AM Ed Cable <[email protected]> > > > wrote: > > > > > > > > > > > > > > > Courage, > > > > > > > > > > > > > > > > I thought that all your notifications work had been merged in > > but > > > > it > > > > > > > > appears it hasn't. Can you update the community on > outstanding > > > > issues > > > > > > to > > > > > > > > address and what review you need from maintainers to complete > > > this? > > > > > > > > > > > > > > > > https://github.com/apache/fineract/pull/405 > > > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > https://issues.apache.org/jira/browse/FINERACT-527 > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > > > Ed > > > > > > > > > > > > > > > > -- > > > > > > > > *Ed Cable* > > > > > > > > President/CEO, Mifos Initiative > > > > > > > > [email protected] | Skype: edcable | Mobile: +1.484.477.8649 > > > > > > > > > > > > > > > > *Collectively Creating a World of 3 Billion Maries | * > > > > > http://mifos.org > > > > > > > > <http://facebook.com/mifos> <http://www.twitter.com/mifos> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > *Ed Cable* > > > > > > President/CEO, Mifos Initiative > > > > > > [email protected] | Skype: edcable | Mobile: +1.484.477.8649 > > > > > > > > > > > > *Collectively Creating a World of 3 Billion Maries | * > > > http://mifos.org > > > > > > <http://facebook.com/mifos> <http://www.twitter.com/mifos> > > > > > > > > > > > > > > > > > > > > > > > -- > > > > *Ed Cable* > > > > President/CEO, Mifos Initiative > > > > [email protected] | Skype: edcable | Mobile: +1.484.477.8649 > > > > > > > > *Collectively Creating a World of 3 Billion Maries | * > http://mifos.org > > > > <http://facebook.com/mifos> <http://www.twitter.com/mifos> > > > > > > > > > > > > -- > Thanks & Regards > > Santosh Math > > *QA Engineer* > > *Conflux Technologies Pvt Ltd <http://www.confluxtechnologies.com/>* > | *Office*: +91-080-41208662 | > > *Address*: #304, 2nd Floor, 7th Main Road, HRBR Layout 1st Block, > Bengaluru, Karnataka, 560043 INDIA >
