CSC/ECE 517 Fall 2021 - E2151.Allow reviewers to bid on what to review: Difference between revisions
Jump to navigation
Jump to search
(Created page with "==Feature== In the current state of implementation the reviews are still assigned using a first-come-first-served basis. Current state of the feature implementation contains b...") |
No edit summary |
||
| Line 3: | Line 3: | ||
==Planned Changes== | ==Planned Changes== | ||
* | * Inspect implementation structure to refactor out duplicated code | ||
* Review ''def self.reviewer_self_topic'' in ''review_bid'' for refacotring to address DRY violation | |||
* Modify ''app/views/review_bids/_all_actions.html.erb'' as fitted to integrate reafacotred controller modules | |||
* Update tests to test the refactored DRYed-out methods | |||
Revision as of 22:23, 2 November 2021
Feature
In the current state of implementation the reviews are still assigned using a first-come-first-served basis. Current state of the feature implementation contains bad code smell and needs improvement to resolve merge conflict and follow DRY principle in order to finalize the implementation. With the current state in mind, the new implementation will enable the capability that reviews could be bid on like how topics are bid on. Behind the scene, the process would involve matching multiple students to review a submission up to the maximum reviewers for a submission.
Planned Changes
- Inspect implementation structure to refactor out duplicated code
- Review def self.reviewer_self_topic in review_bid for refacotring to address DRY violation
- Modify app/views/review_bids/_all_actions.html.erb as fitted to integrate reafacotred controller modules
- Update tests to test the refactored DRYed-out methods