CSC/ECE 517 Fall 2016/E1642. Refactor review response map.rb: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
No edit summary
No edit summary
Line 1: Line 1:
== Peer Review Information ==
For testing the changes made, the following credentials are recommended:
* Instructor Login: '''username:''' ''instructor6'' '''password:''' ''password''
* Student Login: '''username:''' ''student11'' '''password:''' ''password''
* Student Login: '''username:''' ''student1862'' '''password:''' ''password''
The above users are suggested for testing because there are a few users which would lead to exceptions upon login for unknown reasons completely unrelated to our work.
=='''Introduction to Expertiza'''==
=='''Introduction to Expertiza'''==
==='''Background'''===


Expertiza is a peer review system where the students can submit their work and do reviews on fellow students’ work. Expertiza is an open source project and is based on “Ruby on Rails” framework. Expertiza facilitates easy creation of assignments and a list of topics for the same. It allows students to suggest a topic for an assignment as well. Students can form teams and invite other students to join their team for various assignments. Moreover, students can post team advertisements and describe the qualities they are looking for in their team members and other students can respond to the same. Expertiza overall gives a simple web interface for assignment management for both the students as well as instructors.
Expertiza is a peer review system where the students can submit their work and do reviews on fellow students’ work. Expertiza is an open source project and is based on “Ruby on Rails” framework. Expertiza facilitates easy creation of assignments and a list of topics for the same. It allows students to suggest a topic for an assignment as well. Students can form teams and invite other students to join their team for various assignments. Moreover, students can post team advertisements and describe the qualities they are looking for in their team members and other students can respond to the same. Expertiza overall gives a simple web interface for assignment management for both the students as well as instructors.


=='''Background'''==


== Reason for Refractor==
The purpose of this work is to rewrite the code of seven file for several reason listed below.
*Line is too long
*Some code found in other location and should put them in the parent class
*Method has too many lines
*Some of the code will cause security risk
*Using the old style validations
*Using a lower efficent way to do the loop


== Peer Review Information ==
== Peer Review Information ==

Revision as of 22:36, 28 October 2016


Peer Review Information

For testing the changes made, the following credentials are recommended:

  • Instructor Login: username: instructor6 password: password
  • Student Login: username: student11 password: password
  • Student Login: username: student1862 password: password

The above users are suggested for testing because there are a few users which would lead to exceptions upon login for unknown reasons completely unrelated to our work.


Introduction to Expertiza

Background

Expertiza is a peer review system where the students can submit their work and do reviews on fellow students’ work. Expertiza is an open source project and is based on “Ruby on Rails” framework. Expertiza facilitates easy creation of assignments and a list of topics for the same. It allows students to suggest a topic for an assignment as well. Students can form teams and invite other students to join their team for various assignments. Moreover, students can post team advertisements and describe the qualities they are looking for in their team members and other students can respond to the same. Expertiza overall gives a simple web interface for assignment management for both the students as well as instructors.


Reason for Refractor

The purpose of this work is to rewrite the code of seven file for several reason listed below.

  • Line is too long
  • Some code found in other location and should put them in the parent class
  • Method has too many lines
  • Some of the code will cause security risk
  • Using the old style validations
  • Using a lower efficent way to do the loop

Peer Review Information

For users intending to view the deployed Expertiza associated with this assignment, the credentials are below:

  • Instructor login: username -> instructor6, password -> password
  • Student login: username -> student5431, password -> password
  • Student login: username -> student5427, password -> password
few lines with changes