Re: [Dev] Please review and merge the PR[1] for ESBJAVA-3510

2015-05-11 Thread Malaka Silva
Hi Kathees,

Here the fix checked for inbound http and does not show the advance option?

The correct way should be to check if there are no advance parameters don't
show that option?

On Mon, May 11, 2015 at 1:30 PM, Kathees Rajendram kath...@wso2.com wrote:

 Hi,

 $subject.

 [1] - https://github.com/wso2/carbon-mediation/pull/135

 Thanks,
 Kathees

 --
 Kathees
 Software Engineer,
 email: kath...@wso2.com
 mobile: +94772596173




-- 

Best Regards,

Malaka Silva
Senior Tech Lead
M: +94 777 219 791
Tel : 94 11 214 5345
Fax :94 11 2145300
Skype : malaka.sampath.silva
LinkedIn : http://www.linkedin.com/pub/malaka-silva/6/33/77
Blog : http://mrmalakasilva.blogspot.com/

WSO2, Inc.
lean . enterprise . middleware
http://www.wso2.com/
http://www.wso2.com/about/team/malaka-silva/
http://wso2.com/about/team/malaka-silva/

Save a tree -Conserve nature  Save the world for your future. Print this
email only if it is absolutely necessary.
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] Please review and merge the PR[1] for ESBJAVA-3510

2015-05-11 Thread Malaka Silva
Hi Kathees,

UI is written in a generic way. So fix also needs to be done in that manner.

On Mon, May 11, 2015 at 4:19 PM, Kathees Rajendram kath...@wso2.com wrote:

 Hi Malaka,

 The HTTP inbound endpoint does not have the advanced parameters but the
 show/hide icon is in UI. This fix is for removing the icon while HTTP
 endpoint.

 Thanks,
 Kathees

 On Mon, May 11, 2015 at 3:54 PM, Malaka Silva mal...@wso2.com wrote:

 Hi Kathees,

 Here the fix checked for inbound http and does not show the advance
 option?

 The correct way should be to check if there are no advance parameters
 don't show that option?

 On Mon, May 11, 2015 at 1:30 PM, Kathees Rajendram kath...@wso2.com
 wrote:

 Hi,

 $subject.

 [1] - https://github.com/wso2/carbon-mediation/pull/135

 Thanks,
 Kathees

 --
 Kathees
 Software Engineer,
 email: kath...@wso2.com
 mobile: +94772596173




 --

 Best Regards,

 Malaka Silva
 Senior Tech Lead
 M: +94 777 219 791
 Tel : 94 11 214 5345
 Fax :94 11 2145300
 Skype : malaka.sampath.silva
 LinkedIn : http://www.linkedin.com/pub/malaka-silva/6/33/77
 Blog : http://mrmalakasilva.blogspot.com/

 WSO2, Inc.
 lean . enterprise . middleware
 http://www.wso2.com/
 http://www.wso2.com/about/team/malaka-silva/
 http://wso2.com/about/team/malaka-silva/

 Save a tree -Conserve nature  Save the world for your future. Print this
 email only if it is absolutely necessary.




 --
 Kathees
 Software Engineer,
 email: kath...@wso2.com
 mobile: +94772596173




-- 

Best Regards,

Malaka Silva
Senior Tech Lead
M: +94 777 219 791
Tel : 94 11 214 5345
Fax :94 11 2145300
Skype : malaka.sampath.silva
LinkedIn : http://www.linkedin.com/pub/malaka-silva/6/33/77
Blog : http://mrmalakasilva.blogspot.com/

WSO2, Inc.
lean . enterprise . middleware
http://www.wso2.com/
http://www.wso2.com/about/team/malaka-silva/
http://wso2.com/about/team/malaka-silva/

Save a tree -Conserve nature  Save the world for your future. Print this
email only if it is absolutely necessary.
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


[Dev] Please review and merge the PR[1] for ESBJAVA-3510

2015-05-11 Thread Kathees Rajendram
Hi,

$subject.

[1] - https://github.com/wso2/carbon-mediation/pull/135

Thanks,
Kathees

-- 
Kathees
Software Engineer,
email: kath...@wso2.com
mobile: +94772596173
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] Please review and merge the PR[1] for ESBJAVA-3510

2015-05-11 Thread Kathees Rajendram
Hi Malaka,

Ok,I will fix in the generic way.

Thanks
Kathees

On Mon, May 11, 2015 at 4:20 PM, Malaka Silva mal...@wso2.com wrote:

 Hi Kathees,

 UI is written in a generic way. So fix also needs to be done in that
 manner.

 On Mon, May 11, 2015 at 4:19 PM, Kathees Rajendram kath...@wso2.com
 wrote:

 Hi Malaka,

 The HTTP inbound endpoint does not have the advanced parameters but the
 show/hide icon is in UI. This fix is for removing the icon while HTTP
 endpoint.

 Thanks,
 Kathees

 On Mon, May 11, 2015 at 3:54 PM, Malaka Silva mal...@wso2.com wrote:

 Hi Kathees,

 Here the fix checked for inbound http and does not show the advance
 option?

 The correct way should be to check if there are no advance parameters
 don't show that option?

 On Mon, May 11, 2015 at 1:30 PM, Kathees Rajendram kath...@wso2.com
 wrote:

 Hi,

 $subject.

 [1] - https://github.com/wso2/carbon-mediation/pull/135

 Thanks,
 Kathees

 --
 Kathees
 Software Engineer,
 email: kath...@wso2.com
 mobile: +94772596173




 --

 Best Regards,

 Malaka Silva
 Senior Tech Lead
 M: +94 777 219 791
 Tel : 94 11 214 5345
 Fax :94 11 2145300
 Skype : malaka.sampath.silva
 LinkedIn : http://www.linkedin.com/pub/malaka-silva/6/33/77
 Blog : http://mrmalakasilva.blogspot.com/

 WSO2, Inc.
 lean . enterprise . middleware
 http://www.wso2.com/
 http://www.wso2.com/about/team/malaka-silva/
 http://wso2.com/about/team/malaka-silva/

 Save a tree -Conserve nature  Save the world for your future. Print
 this email only if it is absolutely necessary.




 --
 Kathees
 Software Engineer,
 email: kath...@wso2.com
 mobile: +94772596173




 --

 Best Regards,

 Malaka Silva
 Senior Tech Lead
 M: +94 777 219 791
 Tel : 94 11 214 5345
 Fax :94 11 2145300
 Skype : malaka.sampath.silva
 LinkedIn : http://www.linkedin.com/pub/malaka-silva/6/33/77
 Blog : http://mrmalakasilva.blogspot.com/

 WSO2, Inc.
 lean . enterprise . middleware
 http://www.wso2.com/
 http://www.wso2.com/about/team/malaka-silva/
 http://wso2.com/about/team/malaka-silva/

 Save a tree -Conserve nature  Save the world for your future. Print this
 email only if it is absolutely necessary.




-- 
Kathees
Software Engineer,
email: kath...@wso2.com
mobile: +94772596173
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] Please review and merge the PR[1] for ESBJAVA-3510

2015-05-11 Thread Kathees Rajendram
Hi Malaka,

The HTTP inbound endpoint does not have the advanced parameters but the
show/hide icon is in UI. This fix is for removing the icon while HTTP
endpoint.

Thanks,
Kathees

On Mon, May 11, 2015 at 3:54 PM, Malaka Silva mal...@wso2.com wrote:

 Hi Kathees,

 Here the fix checked for inbound http and does not show the advance option?

 The correct way should be to check if there are no advance parameters
 don't show that option?

 On Mon, May 11, 2015 at 1:30 PM, Kathees Rajendram kath...@wso2.com
 wrote:

 Hi,

 $subject.

 [1] - https://github.com/wso2/carbon-mediation/pull/135

 Thanks,
 Kathees

 --
 Kathees
 Software Engineer,
 email: kath...@wso2.com
 mobile: +94772596173




 --

 Best Regards,

 Malaka Silva
 Senior Tech Lead
 M: +94 777 219 791
 Tel : 94 11 214 5345
 Fax :94 11 2145300
 Skype : malaka.sampath.silva
 LinkedIn : http://www.linkedin.com/pub/malaka-silva/6/33/77
 Blog : http://mrmalakasilva.blogspot.com/

 WSO2, Inc.
 lean . enterprise . middleware
 http://www.wso2.com/
 http://www.wso2.com/about/team/malaka-silva/
 http://wso2.com/about/team/malaka-silva/

 Save a tree -Conserve nature  Save the world for your future. Print this
 email only if it is absolutely necessary.




-- 
Kathees
Software Engineer,
email: kath...@wso2.com
mobile: +94772596173
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev