Hi,

Thank You, Isuru...!!

If you add a new property to broker.xml config file, that file needs a
update as well. So broker.xml of carbon-business-messaging [1] has to be
changed I guess. Please add a proper description to the property in the
file for the parameter you are adding.

[1].
https://github.com/wso2/carbon-business-messaging/blob/master/features/andes/org.wso2.carbon.andes.server.feature/resources/conf/broker.xml

Thanks

On Fri, Aug 26, 2016 at 8:15 PM, Imesh Gunaratne <im...@wso2.com> wrote:

> [+ Ramith]
>
> Great work Isuru!! Well written!!
> I think it might be useful for explaining why this feature was needed in
> the PR description.
>
> Thanks!
>
> On Fri, Aug 26, 2016 at 5:37 PM, Isuru Haththotuwa <isu...@wso2.com>
> wrote:
>
>> Hi MB team,
>>
>> Please review the P/R for $subject [1], for the jira ticket reported at
>> [2]. I'm yet to write a test for this, will send a separate P/R once done.
>>
>> [1]. https://github.com/wso2/andes/pull/685
>>
>> [2]. https://wso2.org/jira/browse/MB-1736
>>
>>
>> --
>> Thanks and Regards,
>>
>> Isuru H.
>> +94 716 358 048
>>
>>
>>
>> _______________________________________________
>> Dev mailing list
>> Dev@wso2.org
>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>
>>
>
>
> --
> *Imesh Gunaratne*
> Software Architect
> WSO2 Inc: http://wso2.com
> T: +94 11 214 5345 M: +94 77 374 2057
> W: https://medium.com/@imesh TW: @imesh
> lean. enterprise. middleware
>
>
> _______________________________________________
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 
*Hasitha Abeykoon*
Senior Software Engineer; WSO2, Inc.; http://wso2.com
*cell:* *+94 719363063*
*blog: **abeykoon.blogspot.com* <http://abeykoon.blogspot.com>
_______________________________________________
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to