Closed damithc closed 10 years ago
From dam...@gmail.com on March 01, 2014 01:21:52
Labels: -Priority-Medium -Difficulty-Medium Priority-Low Difficulty-Low SuitableAsFirstIssue
From dam...@gmail.com on March 02, 2014 07:53:29
Hi Yogen, here's an easy issue for you to start with
Owner: in.yo...@gmail.com
From dam...@gmail.com on March 02, 2014 07:55:07
Cc: arnold.k...@gmail.com
Labels: Reviewer-Arnold
From in.yo...@gmail.com on March 05, 2014 07:27:29
Status: Started
From dam...@gmail.com on March 05, 2014 22:03:59
This class doesn't seem to have any warnings. Changing it to a different class.
Summary: EvaluationsLogicTest: address the Eclipse warning (was: StudentEvalSubmissionEditSaveAction: address the Eclipse warning)
From in.yo...@gmail.com on March 06, 2014 00:29:54
I uploaded a patch for the issue. Eclipse warning was due to an unused import. https://codereview.appspot.com/67870055/
Status: ReadyForReview
From arnold.k...@gmail.com on March 06, 2014 00:38:07
Hi Yogen, don;t forget to revert the changes you made for your orientation task (the welcome stranger message at const.java?) and automatic setting changes made by Eclipse (.project and com.google.appengine.eclipse.core.prefs)
Status: ChangesRequested
From in.yo...@gmail.com on March 07, 2014 08:06:22
I have made the changes requested. Reverted my local changes and updated a patch for issue 1656 . https://codereview.appspot.com/72370044/
Status: ReadyForReview
From arnold.k...@gmail.com on March 09, 2014 20:25:27
This issue was updated by revision df19c5c3108a .
Status: Delivered
From dam...@gmail.com on March 14, 2014 23:34:42
Status: Deployed
Labels: Milestone-V4.91
From dam...@gmail.com on March 01, 2014 17:18:02
Eclipse has a warning for this file in the 'Markers' tab. Try to get rid of it.
Original issue: http://code.google.com/p/teammatespes/issues/detail?id=1656