•  

Comment Results

Review Name Created Custom Fields Content
CMRPT-9 12 Nov 2012

these will not be needed as you are going to remove the required validator from the aspx page for this control.

CMRPT-9 14 Nov 2012

changed "txt.branchName" to "txtBranchName"

CMRPT-9 14 Nov 2012

changed "lbl_branchName" to "lblBranchName"

CMRPT-9 14 Nov 2012

deleted

CMRPT-9 14 Nov 2012

changed ddl_emergencyBuild" to "ddlEmergencyBuild"

CMRPT-9 14 Nov 2012

changed lbl_emergencyBuild" to "lblEmergencyBuild"

CMRPT-9 14 Nov 2012

deleted

CMRPT-9 14 Nov 2012

changed "txt_buildDueDate" to txtBuildDueDate"

CMRPT-9 14 Nov 2012

change "lbl_buildDueDate" to "lblBuildDueDate"

CMRPT-9 14 Nov 2012

change "txt_issueID" to "txtIssueID"

CMRPT-9 14 Nov 2012

change "lbl_issueID" to "lblIssueID"

CMRPT-9 14 Nov 2012

removed

CMRPT-9 14 Nov 2012

deleted DropDownList EmergencyBuild

CMRPT-9 14 Nov 2012

deleted

CMRPT-9 14 Nov 2012

deleted

CMRPT-9 14 Nov 2012

changed to "true"

CMRPT-9 14 Nov 2012

updated

CMRPT-9 14 Nov 2012

removed

CMRPT-9 14 Nov 2012

updated

CMRPT-9 14 Nov 2012

removed

CMRPT-9 14 Nov 2012

changed "ddl_reasonCode" to "ddlReasonCode"

CMRPT-9 14 Nov 2012

changed "lbl_reasonCode" to "lblReasonCode"

CMRPT-9 14 Nov 2012

removed

CMRPT-9 14 Nov 2012

updated

CMRPT-9 14 Nov 2012

This is happens in the existing code at other places. I added code to use ex variable, but it'll look bad when exception does happen.

CMRPT-9 14 Nov 2012

updated

CMRPT-9 14 Nov 2012

removed

CMRPT-9 14 Nov 2012

removed

CMRPT-9 14 Nov 2012

Update. I'm wondering why it compiled fine and the tests that I ran also worked. VS did not complain either.

CMRPT-9 14 Nov 2012

updated

CMRPT-9 14 Nov 2012

updated

CMRPT-9 14 Nov 2012

removed

CMRPT-9 14 Nov 2012

removed

CMRPT-9 14 Nov 2012

removed

CMRPT-9 15 Nov 2012

updated

CMRPT-9 15 Nov 2012

updated

CMRPT-9 15 Nov 2012

done

CMRPT-9 15 Nov 2012

done

CMRPT-9 15 Nov 2012

done

CMRPT-9 15 Nov 2012

done

CMRPT-9 15 Nov 2012

done

CMRPT-9 15 Nov 2012

done

CMRPT-9 15 Nov 2012

done

CMRPT-9 15 Nov 2012

updated

CMRPT-9 26 Nov 2012

updated

CMRPT-9 26 Nov 2012

updated

CMRPT-9 26 Nov 2012

updated

CMRPT-9 26 Nov 2012

updated

CMRPT-9 26 Nov 2012

updated

CMRPT-9 26 Nov 2012

updated