CSC/ECE 517 Fall 2021 - E2151.Allow reviewers to bid on what to review

From Expertiza_Wiki
Revision as of 22:23, 2 November 2021 by Wnwang (talk | contribs)
Jump to navigation Jump to search

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