Skip to content
This repository was archived by the owner on Jan 7, 2025. It is now read-only.

Grader feedback fix#72

Open
Searay-330 wants to merge 2 commits into
devfrom
grader_feedback_fix
Open

Grader feedback fix#72
Searay-330 wants to merge 2 commits into
devfrom
grader_feedback_fix

Conversation

@Searay-330

Copy link
Copy Markdown
Collaborator

No description provided.

@Searay-330
Searay-330 requested a review from daviddob June 13, 2020 16:07
@Searay-330 Searay-330 self-assigned this Jun 13, 2020

@daviddob daviddob left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fix likely needs to be applied in a few additional places.

Comment thread app/models/submission.rb
if assessment.is_section_dependant
form_hash["section"] = (assessment.lecture?) ? course_user_datum.lecture : course_user_datum.section
end
form_hash["ip"] = self.submitter_ip

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This change is part of a separate change that has been merged - it should not be included here

Score for this problem: <%= @score.score.to_f().round(1) %>

Graded by: <%= if @score.grader && @score.grader.user then @score.grader.user.email end %>
Graded by: <%= if @score.grader && @score.grader.user then @score.grader.user.email elsif @assessment.has_autograder? then "Autograder" end %>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There are other places this needs to be resolved as well - for example:

Graded by: <%= if @score.grader then @score.grader.email end %>

Graded by: <%= if @score.grader then @score.grader.email end %>

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.

2 participants