https://codereview.appspot.com/72980044/diff/1/src/main/webapp/js/instructorStudentRecords.js File src/main/webapp/js/instructorStudentRecords.js (right): https://codereview.appspot.com/72980044/diff/1/src/main/webapp/js/instructorStudentRecords.js#newcode37 src/main/webapp/js/instructorStudentRecords.js:37: { '{' should be in the previous line.
10 years, 6 months ago
(2014-03-09 12:41:19 UTC)
#1
https://codereview.appspot.com/72980044/diff/70001/src/test/resources/pages/InstructorCourseDetailsPage.html File src/test/resources/pages/InstructorCourseDetailsPage.html (left): https://codereview.appspot.com/72980044/diff/70001/src/test/resources/pages/InstructorCourseDetailsPage.html#oldcode128 src/test/resources/pages/InstructorCourseDetailsPage.html:128: <a onmouseout="hideddrivetip()" onmouseover="ddrivetip('Delete the student and the corresponding evaluations ...
10 years, 6 months ago
(2014-03-24 11:48:11 UTC)
#3
https://codereview.appspot.com/72980044/diff/70001/src/test/resources/pages/I...
File src/test/resources/pages/InstructorCourseDetailsPage.html (left):
https://codereview.appspot.com/72980044/diff/70001/src/test/resources/pages/I...
src/test/resources/pages/InstructorCourseDetailsPage.html:128: <a
onmouseout="hideddrivetip()" onmouseover="ddrivetip('Delete the student and the
corresponding evaluations from the course')" onclick="return
toggleDeleteStudentConfirmation('Alice Betsy')"
href="/page/instructorCourseStudentDelete?courseid=CCDetailsUiT.CS2104&studentemail={$test.student1}%40gmail.com&user=CCDetailsUiT.instr"
class="color_black t_student_delete0">
Karan, Note the {$test.student1} above. This is a placeholder for our test
script to inject the test account ID, based on the test.properties file. If not,
the test will fail when the test account id is changed by a developer (each
developer uses a different account).
Issue 72980044: Issue 1685 : instructorCourseView: add 'comment' link to each student
Created 10 years, 6 months ago by KaranKamath26
Modified 10 years, 6 months ago
Reviewers: damith, jamesdjuhartono
Base URL:
Comments: 4