Hi Chamila, I'll go through that.thanks for the detail.
Regards Krishan. On Wed, Aug 9, 2017 at 10:15 AM, Chamila Adhikarinayake <[email protected]> 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 <[email protected]> > wrote: > >> Hi Ishara, >> >> Please find my answers in-line. >> >> >> >> On Fri, Aug 4, 2017 at 11:35 AM, Ishara Cooray <[email protected]> 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 : [email protected] >> 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 - [email protected] > Blog - http://helpfromadhi.blogspot.com/ > -- *Krishan Wijesena* Software Engineer | WSO2 Email : [email protected] Mobile : +94776219923 WSO2 Inc : http://wso2.com [image: http://wso2.com/signature] <http://wso2.com/signature>
_______________________________________________ Architecture mailing list [email protected] https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture
