[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-29 Thread ad-os
Github user ad-os closed the pull request at: https://github.com/apache/incubator-fineract/pull/270 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-fineract pull request #283: Notification Module

2017-02-27 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/283#discussion_r103187242 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/domain/Notification.java --- @@ -0,0 +1,112

[GitHub] incubator-fineract pull request #283: Notification Module

2017-02-27 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/283#discussion_r103213061 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/service/NotificationGeneratorReadPlatformServiceImpl.java --- @@ -0,0

[GitHub] incubator-fineract pull request #283: Notification Module

2017-02-27 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/283#discussion_r103218666 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/eventandlistener/NotificationEvent.java --- @@ -0,0 +1,50

[GitHub] incubator-fineract pull request #283: Notification Module

2017-02-27 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/283#discussion_r103217772 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/service/NotificationMapperReadPlatformServiceImpl.java --- @@ -0,0

[GitHub] incubator-fineract issue #178: Added notification module

2016-11-22 Thread ad-os
Github user ad-os commented on the issue: https://github.com/apache/incubator-fineract/pull/178 Okay, will do and get back to you :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] incubator-fineract issue #178: Added notification module

2016-11-28 Thread ad-os
Github user ad-os commented on the issue: https://github.com/apache/incubator-fineract/pull/178 Hi Nazeer, What do you mean by > API resourced should not have this changes ? What I have currently refactored is like initiating the business events in the respective

[GitHub] incubator-fineract issue #178: Added notification module

2016-11-17 Thread ad-os
Github user ad-os commented on the issue: https://github.com/apache/incubator-fineract/pull/178 I have used jms to publish the generated notifications to a queue(implmented via ActiveMq) and then used the jmslistener to subscribe to that queue and listen for any new notification

[GitHub] incubator-fineract issue #178: Added notification module

2016-11-17 Thread ad-os
Github user ad-os commented on the issue: https://github.com/apache/incubator-fineract/pull/178 I think I have followed the same design as the implementation of business events. > You no need to modify all API resources I have used only the publish method call f

[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-13 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/270#discussion_r96107755 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/eventandlistener/NotificationEventListener.java --- @@ -0,0 +1,101

[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-13 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/270#discussion_r96110571 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/domain/NotificationGenerator.java --- @@ -0,0 +1,178

[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-12 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/270#discussion_r95950094 --- Diff: fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/NotificationApiTest.java --- @@ -0,0 +1,57

[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-13 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/270#discussion_r95986979 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/domain/NotificationMapper.java --- @@ -0,0 +1,121

[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-13 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/270#discussion_r96020577 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/domain/NotificationGenerator.java --- @@ -0,0 +1,178

[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-13 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/270#discussion_r96020630 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/domain/NotificationMapper.java --- @@ -0,0 +1,121

[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-13 Thread ad-os
Github user ad-os commented on a diff in the pull request: https://github.com/apache/incubator-fineract/pull/270#discussion_r96020429 --- Diff: fineract-provider/src/main/java/org/apache/fineract/notification/data/NotificationData.java --- @@ -0,0 +1,220

[GitHub] incubator-fineract pull request #270: Notification Module

2017-01-03 Thread ad-os
GitHub user ad-os opened a pull request: https://github.com/apache/incubator-fineract/pull/270 Notification Module This module will allow developers to integrate notifications with their functionality. Here are some wikis on which you can read more about this module. 1

[GitHub] incubator-fineract pull request #178: Added notification module

2017-01-02 Thread ad-os
Github user ad-os closed the pull request at: https://github.com/apache/incubator-fineract/pull/178 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so