Skip to content

Commit

Permalink
Resolve approach to proposal notification
Browse files Browse the repository at this point in the history
  • Loading branch information
mghaught committed Nov 29, 2018
1 parent dc04aac commit b4ff3fb
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions app/controllers/staff/proposals_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -82,13 +82,7 @@ def finalize
authorize @proposal, :finalize?

if @proposal.finalize
# FinalizationNotifier.notify(@proposal)
Staff::ProposalMailer.send_email(@proposal).deliver_now
@proposal.speakers.map(&:user).each do |user|
Notification.create_for(user, proposal: @proposal,
message: "Your proposal for #{@proposal.event.name} has been #{@proposal.state}",
target_path: finalized_notification_event_proposal_url(@proposal.event, @proposal))
end
FinalizationNotifier.notify(@proposal)
else
flash[:danger] = "There was a problem finalizing the proposal: #{@proposal.errors.full_messages.join(', ')}"
end
Expand Down

0 comments on commit b4ff3fb

Please sign in to comment.