Hi Chamila,

I'll go through that.thanks for the detail.

Regards
Krishan.

On Wed, Aug 9, 2017 at 10:15 AM, Chamila Adhikarinayake <chami...@wso2.com>
wrote:

> Hi Krishan,
>
> There was an intern project done to introduce Event Listeners using
> observers to api manager. I think you could re-use this feature for your
> implementation as well. There is a simple listener added to log message for
> api creation (see [1][2]) . You might be able to use the same mechanism for
> this feature as well when sending emails.
>
> [1] https://github.com/wso2/carbon-apimgt/blob/master/
> components/apimgt/org.wso2.carbon.apimgt.core/src/main/
> java/org/wso2/carbon/apimgt/core/impl/APIPublisherImpl.java#L335
> [2] https://github.com/wso2/carbon-apimgt/blob/master/
> components/apimgt/org.wso2.carbon.apimgt.core/src/main/
> java/org/wso2/carbon/apimgt/core/impl/EventLogger.java
>
> Regards
> Chamila.
>
> On Fri, Aug 4, 2017 at 12:06 AM, Krishan Wijesena <krish...@wso2.com>
> wrote:
>
>> Hi Ishara,
>>
>> Please find my answers in-line.
>>
>>
>>
>> On Fri, Aug 4, 2017 at 11:35 AM, Ishara Cooray <isha...@wso2.com> wrote:
>>
>>> Hi Krishan,
>>>
>>> I guess you are planing to add this configuration into deployment.yaml
>>> which is the Global configuration model in C5.
>>>
>>> If that is the case how will it going to be in the deployment.yaml?
>>> All the required configurations should define as notification
>>> configuration java bean class and create its object in APIM configuration
>>> global bean class so it can generate
>>>
>>     configuration file when build component later.
>>
>>
>>> Do you plan to add it into a separate name space or under same "
>>> wso2.carbon.apimgt" namespace?
>>>
>> As our practice i guess it is better if you add notification
>>> configuration in here so that you can add your bean class in apimgt.core
>>>
>>    No Im not planning any separate name space.just adding to
>> "wso2.carbon.apimgt"
>>
>>>
>>> NotificationTypeConfiguration class hold “Type” which needs to send
>>> notification and “NotifierConfiguration” object.
>>>
>>> For example “Subscription”, “New Api Version” etc.
>>>
>>>
>>> when you say Type, i felt like it's the mode of notification such as
>>> email, sms
>>>
>>> So how about changing it to something like 'Subject' or anything
>>> suitable?
>>>
>>     Actually Notification type refers to purpose of sending
>> notification.
>>     For example New Api version create notification can send as Email
>> ,SMS etc.
>>     I'll change that in to more suitable form thanks for mentioning it.
>>
>>>
>>>
>>>
>>>
>>> Thanks & Regards,
>>> Ishara Cooray
>>> Senior Software Engineer
>>> Mobile : +9477 262 9512 <+94%2077%20262%209512>
>>> WSO2, Inc. | http://wso2.com/
>>> Lean . Enterprise . Middleware
>>>
>>>
>>>> Thanks and Regards
>>
>> --
>>
>> *Krishan Wijesena*
>> Software Engineer | WSO2
>>
>> Email : krish...@wso2.com
>> Mobile : +94776219923 <+94%2077%20621%209923>
>> WSO2 Inc : http://wso2.com
>> [image: http://wso2.com/signature] <http://wso2.com/signature>
>>
>
>
>
> --
> Regards,
> Chamila Adhikarinayake
> Senior Software Engineer
> WSO2, Inc.
> Mobile - +94712346437 <+94%2071%20234%206437>
> Email  - chami...@wso2.com
> Blog  -  http://helpfromadhi.blogspot.com/
>



-- 

*Krishan Wijesena*
Software Engineer | WSO2

Email : krish...@wso2.com
Mobile : +94776219923
WSO2 Inc : http://wso2.com
[image: http://wso2.com/signature] <http://wso2.com/signature>
_______________________________________________
Architecture mailing list
Architecture@wso2.org
https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture

Reply via email to