Macro to email based on Renewal Date

[howard]

I have written code to email a recipient where "yes" appears in Col G. Where "Yes" Appears in Col "G", then I want the header as well as the rows "Yes" to be emailed to the recipient"


I cannot get the code to do this


See full code below

 Sub Email_Reminder()
    Dim Email_Subject As String, Email_Send_To As String, Email_Body As String
    Dim Mail_Object As Object, Mail_Single As Variant
    Dim r As Long
    Dim cell As Range

r = 2

Do Until Trim(Cells(r, 7).Value) = ""
    Email_Subject = Sheets("Email").Range("B1")
    Email_Send_To = Cells(r, 6).Value
    Email_Body = Sheets("Email").Range("B2")

    For Each cell In Columns("G:G")

        If cell.Value = "Yes" Then

            On Error GoTo debugs
            Set Mail_Object = CreateObject("Outlook.Application")
            Set Mail_Single = Mail_Object.CreateItem(0)
            With Mail_Single
                .Subject = Email_Subject
                .To = Email_Send_To
                .Body = Email_Body
                .Display
            End With

        End If
ResumeLoop:
    Next cell
Loop

Exit Sub
debugs:
    If Err.Description <> "" Then MsgBox Err.Description
    GoTo ResumeLoop:
End Sub

See link for sample data


https://www.dropbox.com/s/rjqsnkgpb8gj19p/Email Reminder based on Date.xlsm?dl=0

It would be appreciated if someone could assist me

I have also posted on link below

https://www.excelforum.com/excel-pr...omaticaly-send-email-based-on-renew-date.html

 

[DanteAmor]

Dante I have one more request

Once Email has been generated, I would like to insert "Emailed" in Col I in the same row as the items that were emailed

It would be appreciated if you could incorporate this in your code if this is at all possible

Try this

Sub Email_Reminder()
    Dim Mail_Single As Variant, c As Range, wFile As String
    Dim sh As Worksheet, shE As Worksheet, s As Worksheet, wb2 As Workbook
    
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False
    Set sh = Sheets("Renewals")
    Set shE = Sheets("Email")
    On Error Resume Next
    For Each c In sh.Range("G2", Range("G" & Rows.Count).End(xlUp))
        If c.Value = "Yes" Then
            For Each s In Sheets
                If UCase(s.Name) = UCase(sh.Cells(c.Row, "A").Value) Then
                    s.Copy
                    Set wb2 = ActiveWorkbook
                    wFile = ThisWorkbook.Path & "\" & s.Name & ".xlsx"
                    wb2.SaveAs wFile
                    Set Mail_Single = CreateObject("Outlook.Application").CreateItem(0)
                    With Mail_Single
                        .Subject = shE.Range("B1").Value
                        .To = sh.Cells(c.Row, "F").Value
                        .Body = shE.Range("B2").Value
                        .Attachments.Add wFile
                        .Display
                    End With
                    [COLOR=#0000ff]sh.Cells(c.Row, "I").Value = "Emailed"[/COLOR]
                    wb2.Close False
                    Exit For
                End If
            Next
        End If
    Next
End Sub

 

[howard]

Hi Dante

Thanks very much this is perfect

 

[DanteAmor]

Hi Dante

Thanks very much this is perfect

I'm glad to help you. Thanks for the feedback.

 

[howard]

Hi Dante


I checked your data again today and 4 emails are generated and only 3 should be generated

Kindly test and amend code


https://www.dropbox.com/s/rjqsnkgpb8gj19p/Email Reminder based on Date.xlsm?dl=0

 

[DanteAmor]

Hi Dante

I checked your data again today and 4 emails are generated and only 3 should be generated
Kindly test and amend code
https://www.dropbox.com/s/rjqsnkgpb8gj19p/Email Reminder based on Date.xlsm?dl=0

If the cell says "yes" then an email is generated

 

[howard]

I amended your code slightly that that if "yes" appears in Col G & "emailed" appears in Col I not to send an email


Kindly test & amend

 

[DanteAmor]

I amended your code slightly that that if "yes" appears in Col G & "emailed" appears in Col I not to send an email


Kindly test & amend

So, does it work?

 

[howard]

Hi Dante

Your original code worked 100% , but I added an additional Criteria as follows:

1) If Col G = "Yes" and Col I = "NO" or Blank , then generate an email using recipient/s in Col F in same row as criteria
2) If Col G = "Yes" and Col I = "Emailed" no email to be generated
3) If Col G = "No' then no email to be generated


See my code below

 Sub email()

Dim Mail_Single As Variant, c As Range, d As Range, wFile As String
    Dim sh As Worksheet, shE As Worksheet, s As Worksheet, wb2 As Workbook
    
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False
    Set sh = Sheets("Renewals")
    Set shE = Sheets("Email")
    On Error Resume Next
    For Each c In sh.Range("G2", Range("G" & Rows.Count).End(xlUp))
    For Each d In sh.Range("I2", Range("I" & Rows.Count).End(xlUp))
        If c.Value = "Yes" And d.Value <> "Emailed" Then
            For Each s In Sheets
                If UCase(s.Name) = UCase(sh.Cells(c.Row, "A").Value) Then
                    s.Copy
                    Set wb2 = ActiveWorkbook
                    wFile = ThisWorkbook.Path & "\" & s.Name & ".xlsx"
                    wb2.SaveAs wFile
                    Set Mail_Single = CreateObject("Outlook.Application").CreateItem(0)
                    With Mail_Single
                        .Subject = shE.Range("B1").Value
                        .To = sh.Cells(c.Row, "F").Value
                        .Body = shE.Range("B2").Value
                        .Attachments.Add wFile
                        .Display
                    End With
                    sh.Cells(c.Row, "I").Value = "Emailed"
                    wb2.Close False
                    Exit For
                End If
            Next
        End If
    Next
     Next
End Sub

See link to workbook on Dropbox



https://www.dropbox.com/s/rjqsnkgpb8gj19p/Email Reminder based on Date.xlsm?dl=0

 

[DanteAmor]

Hi Dante

Your original code worked 100% , but I added an additional Criteria as follows:

1) If Col G = "Yes" and Col I = "NO" or Blank , then generate an email using recipient/s in Col F in same row as criteria
2) If Col G = "Yes" and Col I = "Emailed" no email to be generated
3) If Col G = "No' then no email to be generated


See my code below

See link to workbook on Dropbox

https://www.dropbox.com/s/rjqsnkgpb8gj19p/Email Reminder based on Date.xlsm?dl=0

Ok, you added more criteria, but it works, or I do not understand why the file

 

[howard]

Hi Dante

If for eg G2 = "Yes" and I2 = "NO" or Blank, the macro creates 2 emails. It should only create 1 email


Please test and amend code