Home / Questions / 111675
Accepted
juan gomez
Asked: 2020-10-24 11:01:11 +0800 CST 2020-10-24 11:01:11 +0800 CST

Send more than one email in the same procedure

  • 772

How do I get it to send me the first email? it only sends the last one and i want it to send both of them.

In the email and emailsoli fields, each field has a saved email.

class ReunionMailer < ApplicationMailer
   default from: '[email protected]' 
   def autorizar_email(reunion)
      @reunion = reunion
      @url  = 'http://example.com/login'
      mail(to: @reunion.email , subject: 'Autorización de aprovador')
      mail(to: @reunion.emailsoli , subject: 'Autorización de aprovador')
   end
end

meeting.erb

def  autorizar
  if self.auto = true 
    ReunionMailer.autorizar_email(Reunion.find(self.id)).deliver_now 

  end
end
ruby-on-rails

2 Answers

  1. You must send a arraywith the necessary email addresses

    def autorizar_email(reunion)
  @reunion = reunion
  @url  = 'http://example.com/login'
  @emails = [@reunion.email]
  @emails << @reunion.emailsoli
  mail(to: @emails, subject: 'Autorización de aprovador')
end
    • 0
  2. Best Answer

    In case you want to send a single email in which both recipients are included, the jvilla8a solution is the right one.

    If what you want is to send two emails (one for each address) then you must call the method autorizar_emailtwice (once for each email); for instance:

    meeting_mailer.rb:

    class ReunionMailer < ApplicationMailer
  default from: '[email protected]' 

  def autorizar_email(reunion, email)
     @reunion = reunion
     @url  = 'http://example.com/login'
     mail(to: email , subject: 'Autorización de aprovador')
  end
end

    meeting.rb:

    def  autorizar
  if auto
    reunion = Reunion.find(id)

    reunion.slice(:email, :emailsoli).values.each do |email|
      ReunionMailer.autorizar_email(reunion, email).deliver_now
    end 
  end
end

    You will notice that I have removed the use of selfsince it is not necessary to use it when you want to get the value of some attribute (it is necessary if you are looking to change the value).

    I also changed auto = true(which I imagine you were really looking to use auto == true) simply to auto(which already returns some value trueor false).

    • 0