Skip to content
This repository has been archived by the owner on Dec 10, 2017. It is now read-only.

Kevin/exams front end #46

Merged
merged 10 commits into from
Jan 24, 2014
Merged

Kevin/exams front end #46

merged 10 commits into from
Jan 24, 2014

Conversation

kacasey8
Copy link
Contributor

Studrel admin should be legit, studrel can edit and delete exams now. Exams.semester and exams.year shouldn't exist b/c exams belong to a course offering which knows what semester it is in, but leaving it in for now in case anyone disagrees. Exams index should hold similar functionality to old website


@counts = Hash.new(0)
Exam.includes(course_offering: :course).each do |x|
@counts[x.course_offering.course.id] += 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since you do courses.each in the view anyways, you can just use course.exams.count instead of this i believe

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

resolved

alancyao added a commit that referenced this pull request Jan 24, 2014
@alancyao alancyao merged commit 0a3936a into master Jan 24, 2014
@kacasey8 kacasey8 deleted the kevin/exams_front_end branch January 27, 2014 00:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants