Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

E1413 Reconcile View Scores and Review Report and Improve scores views #410

Merged
merged 12 commits into from
Sep 25, 2014

Conversation

MehakMehta
Copy link

Added two new views and extracted scores using single queries.
Files changed:
config/routes.rb ->added routes for the below added views
app/models/assignment.rb ->refactored get_scores method
app/controllers/grades_controller.rb ->added two new methods, all_scores and view_course_scores
Files added:
app/views/all_scores.html.erb ->displays scores of all assignments for a user
app/views/view_course_scores ->displays scores for all assignment and users for a course

Added options for viewing Teammate review report, author's feedback remote and metareview report
Changed pops to normal html pages
Fixed bug for displaying the details of reviews given by a reviewer to show the details of all reviews given instead of just one.

MehakMehta and others added 12 commits March 31, 2014 20:25
Added two new views and extracted scores using single queries
merge branch 'master' of git://github.com/makhavne/expertiza into makhavne-master

Conflicts:
	app/models/assignment_team.rb
    merge branch 'master' of git://github.com/makhavne/expertiza into master
the set of candidate submissions to review.
Added options for Metareview report, Author’s Feedback Report and
Teammate Review Report.
Changed from popups to normal htmls
Displayed details of all reviews done by a reviewer of a particular
type instead of just one.
@MehakMehta MehakMehta changed the title Improve scores views E1413 Reconcile View Scores and Review Report and Improve scores views Apr 25, 2014
@krunaljhaveri krunaljhaveri reopened this Sep 25, 2014
@krunaljhaveri krunaljhaveri merged commit 7ce59a6 into expertiza:master Sep 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants