CSC/ECE 517 Fall 2016/E1641. Refactor review mapping controller.rb

From Expertiza_Wiki
Revision as of 01:37, 29 October 2016 by Jdeng8 (talk | contribs) (Created page with "==Background== ==Current Implementation and Problems== 4. There was a self-review feature, the method add_self_reviewr, get_team_from_submission are related to this. Two views c...")
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to navigation Jump to search

Background

Current Implementation and Problems

4. There was a self-review feature, the method add_self_reviewr, get_team_from_submission are related to this. Two views calls add_self_reviewer are ç. The names of views are not related to self_review feature. Plus those two views are not called anywhere.

Changes Implemented

4. By searching the whole project and routes, we verify that methods add_self_reviewr and get_team_from_submission in this controller are not called by any other methods except for views add_self_reviewr, get_team_from_submission. And those two views are not linked to any other views. So they are deleted from the project.

Test