Xiao Pu - Weekly Progress

Week 2

Effort	Type	PR	Issue
2	D	`#6324`	Show “Participants who have not responded to any question” panel in grey
4	D	`#6344`	Implement InstructorFeedbackRemind(ParticularStudents)ActionTest

Week 3

Effort	Type	PR	Issue
1	D	`#6213`	id=”caption” in static pages should be class instead #6190
1	D	`#6352`	Moderating responses/previewing sessions heading does not properly escape HTML characters #5932
1	D	`#6414`	Create another InstructorEditStudentFeedbackPage page objects for testing #6396
4	D	`#6352`	Instructor: moderate feedback: give message “Some questions may be hidden…” as a status message #2818

Week 4

Effort	Type	PR	Issue
1	D	`#6413`	Refactor test cases code in InstructorFeedbackResultsPageUiTest and StudentFeedbackResultsPageUiTest #6410
2	D	`#6444`	Instructor view session results: numeric scale question: sorting is incorrect #6437
2	D	`#6375`	Show loading gif picture while the profile picture is being loaded #2312

Week 5

Effort	Type	PR	Issue
4	D	`#6382`	instructorFeedbackSessionAdd form: Give an edit links to both sentences #2305
1	R	`#6517`	InstructorCourseRemindActionTest: refactor duplicated code #6516

Other work:

Effort	Work
4	Study CheckStyle and propose additional CheckStyle rule for teammates `#6519`

Week 6

Effort	Type	PR	Issue
2	D	`#6600`	Add CommentSearchTest to test searching for comments #6549
4	D	`#6546`	instructorStudentList: add a ‘mailto’ link under the list of emails #1682

Week 7

Effort	Type	PR	Issue
2	D	`#6628`	InstructorStudentListPageUiTest failing on live server #6620
4	R	`#6577`	Add InstructorSearchTest to test searching for instructors #6551
4	R	`#6652`	Fix some violations reported by FindBugs #6646

Week 8

Effort	Type	PR	Issue
4	D	`#6655`	Implement AdminActivityLogPageActionTest #6654
2	D	`#6686`	Bug: Student received submission confirmation but the responses are not recorded #6643
1	R	`#6667`	Disallow blank lines containing whitespace #6666

Week 9

Effort	Type	PR	Issue
4	D	`#6524`	Separate the UI part and logic part in ActivityLogEntry #6503
2	D	`#6778`	Random test fail in *LogicTest.java recently #6777
2	R	`#6583`	Add Javadoc rules to Checkstyle without full enforcement #6582
1	R	`#6804`	Add instruction to install eslint-plugin-json for eslint #6802

Week 10

Effort	Type	PR	Issue
4	D	`#6755`	Separate the UI and logic for EmailLogEntry #6753
2	D	`#6814`	Add CheckStyle rule CommentsIndentation #6813
1	D	`#6793`	Codecov missing base report #6791
2	R	`#6897`	Add more Javadoc to the code base #6892
1	R	`#6853`	Different types of styles/fonts used in staticAnalysis.md #6843
1	R	`#6856`	Navbar showing contents behind it #6823

Week 11

Effort	Type	PR	Issue
4	O	CS2103R Report	Additional Static Analysis Rules for TEAMMATES

Week 12

Effort| Type | PR | Issue :—-:|:—-:|:———–|:—— 4 | D | #6876 | Implement AdminEmailLogPageActionTest #6855 2 | R | #6669 | Failed to instantiate Feedback*QuestionDetails instance for CONTRIB question type #6619 2 | R | #7051 | AdminEmailComposeSendAction has no log message on success #6699 1 | R | #7029 |Trivial formatting error in design.md #7009 1 | R | #6863 | Remove useless replacement of text in AdminActivityLogPageActionTest #6857 1 | R | #6858 | Make use of StringHelper.join() in ActivityLogEntry and AdminActivityLogPageData #6844 2 | O | CS2103R Report | Revise Report: Technical Report: Static Analysis(Java) in TEAMMATES

Week 13

Effort| Type | PR | Issue :—-:|:—-:|:———–|:—— 2 | R | #7126 |Add more import rules to CheckStyle #7125 4 | O | SoC 10th STePS | Attend 10th STePS project showcase 4 | O | #7132 | Fix some violations to boolean variable naming convention #7131 2 | O | #7090 | Add more rules to Macker to enforce architectural rules #7086