Hi Peter, We will take a look at this and keep you informed thanks On 3 September 2013 23:10, Peter Palaga <ppalaga(a)redhat.com <mailto:ppalaga@redhat.com>> wrote: Hi, could please somebody from WebUI people look at this [1] change set? It is correct and harmless as far as I can see, but someone more experienced might see it differently. Thanks, Peter [1] https://github.com/gatein/gatein-portal/pull/644 _______________________________________________ gatein-dev mailing list gatein-dev(a)lists.jboss.org <mailto:gatein-dev@lists.jboss.org> https://lists.jboss.org/mailman/listinfo/gatein-dev -- *Trong Tran* /(+84) 983841909 | /trongtt(a)gmail.com <mailto:trongtt@gmail.com> Twitter: http://twitter.com/trongtt//

Keepalive? -- PP On 2013-09-04 09:05, Peter Palaga wrote: > Thanks, Trong! -- PP > > On 2013-09-04 09:01, Trong Tran wrote: > >> Hi Peter, >> >> We will take a look at this and keep you informed >> >> thanks >> >> >> On 3 September 2013 23:10, Peter Palaga <ppalaga(a)redhat.com >> <mailto:ppalaga@redhat.com>> wrote: >> >> Hi, >> >> could please somebody from WebUI people look at this [1] change >> set? It >> is correct and harmless as far as I can see, but someone more >> experienced might see it differently. >> >> Thanks, >> >> Peter >> >> [1] https://github.com/gatein/**gatein-portal/pull/644<https://github.com/... >> ______________________________**_________________ >> gatein-dev mailing list >> gatein-dev(a)lists.jboss.org <mailto:gatein-dev@lists.**jboss.org<gatein-dev@lists.jboss.org> >> > >> https://lists.jboss.org/**mailman/listinfo/gatein-dev<https://lists.jb... >> >> >> >> >> -- >> *Trong Tran* >> /(+84) 983841909 | /trongtt(a)gmail.com <mailto:trongtt@gmail.com> >> Twitter: http://twitter.com/trongtt// >> > > ______________________________**_________________ > gatein-dev mailing list > gatein-dev(a)lists.jboss.org > https://lists.jboss.org/**mailman/listinfo/gatein-dev<https://lists.jb... > >

Hi Peter, The PR looks OK to merge. Thanks for your fix. On 3 September 2013 23:10, Peter Palaga <ppalaga(a)redhat.com <mailto:ppalaga@redhat.com>> wrote: Hi, could please somebody from WebUI people look at this [1] change set? It is correct and harmless as far as I can see, but someone more experienced might see it differently. Thanks, Peter [1] https://github.com/gatein/gatein-portal/pull/644 _______________________________________________ gatein-dev mailing list gatein-dev(a)lists.jboss.org <mailto:gatein-dev@lists.jboss.org> https://lists.jboss.org/mailman/listinfo/gatein-dev -- *Trong Tran* /(+84) 983841909 | /trongtt(a)gmail.com <mailto:trongtt@gmail.com> Twitter: http://twitter.com/trongtt//