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 <avikganguly...@gmail.com> 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 <couragean...@gmail.com> > 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 <edca...@mifos.org> 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 <couragean...@gmail.com> > > > 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 <edca...@mifos.org> 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 < > > couragean...@gmail.com> > > > > > 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 <edca...@mifos.org> > > 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 > > > > > > > edca...@mifos.org | 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 > > > > > edca...@mifos.org | 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 > > > edca...@mifos.org | 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