Dumindu, Please provide a deadline by which you will be able to complete the enhancements that we discussed during the code review.
Azeez On Thu, Oct 29, 2009 at 4:28 AM, ratha <[email protected]> wrote: > > > Hi all, > > Following notes were taken from today's session. > > 1. Merge *BAM.List* and *BAM.Mgt *bundles with core bundle. > > * Have only org.wso2.carbon.bam and org.wso2.carbon.bam.ui bundles. > > 2.BAM.Util--->Constants class should go under core bundle. > > 3. Mgt.UI > > * pom.xml should not have a <fragment-host> element. Export all the > bundles with *carbon ui *header. > * i18n all the messages not only in JSPs but also in client classes. > * BAMStatqueryDsClient class need to be separated. > * BackendServiceURL should point to the service. Remove the > operation from ServiceURL in the client class. > * Write common client class and share among all other bundles.( > gauges&mgt.ui share common client classes) > * BAMStatQueryDsclient class has two constructors. Need to remove. > * JSPs should handle exception properly. > o Before rendering the page, do all the remote calls. > o In websphere after deleting a server redirection to the > server list page can be done through CarbonUI.utils--->get > context root. > o Set the error status to 500 . > o forward the errors to carbon/error.jsp. > > Thanks. > -Ratha. > > > _______________________________________________ > Carbon-dev mailing list > [email protected] > https://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev > > -- -- Afkham Azeez [email protected] WSO2 Inc. http://wso2.com Blog: http://afkham.org
_______________________________________________ Carbon-dev mailing list [email protected] https://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
