Skip to content

Commit

Permalink
Fixing Review Comments
Browse files Browse the repository at this point in the history
  • Loading branch information
Shounaks committed Dec 8, 2024
1 parent fcabcf4 commit 5fd2a38
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion app/mailers/teacher_response_mailer.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ def add_general
@unsubscribe_url = "#{@doubtfire_host}/#/home?notifications"
end

def recieved_notification(project, task)
def received_notification(project, task)
return nil if project.nil?

add_general
Expand Down
4 changes: 2 additions & 2 deletions app/models/task.rb
Original file line number Diff line number Diff line change
Expand Up @@ -738,9 +738,9 @@ def add_status_comment(current_user, status)
# send emails...
begin
logger.info "Sending comment notification email for project #{project.id}"
TeacherResponseMailer.recieved_notification(project,self).deliver
TeacherResponseMailer.received_notification(project,self).deliver
rescue => e
logger.error "Failed to send emails from comment submission. Rescued with error: #{e.message}"
logger.error "Failed to send emails from comment submission.ProjectID: #{project.id} for task: #{self.id} . Rescued with error: #{e.message}"
end

comment
Expand Down

0 comments on commit 5fd2a38

Please sign in to comment.