Sending email when a Cell is changed to value "A" for multiple colums

[Sharonca]

Hello,
I have a macro in a shared sheet (different users may be updating different cells in the same column at the same time) that currently works if a cell in the designated range changes to any value. I would need to change that to only when the value changes to "A". I also need to have the same macro for multiple columns. I don't know if I need to put the code for each column and call the module or if there is another solution.

Here is the code I have thus far.

Private Sub Worksheet_Change(ByVal Target As Range)
Dim xRgSel As Range
Dim xOutApp As Object
Dim xMailItem As Object
Dim xMailBody As String
On Error Resume Next
Application.ScreenUpdating = False
Application.DisplayAlerts = False
Set xRg = Range("AA4:AA2000")
Set xRgSel = Intersect(Target, xRg)
ActiveWorkbook.Save
If Not xRgSel Is Nothing Then
Set xOutApp = CreateObject("Outlook.Application")
Set xMailItem = xOutApp.CreateItem(0)
xMailBody = "Cell(s) " & xRgSel.Address(False, False) & _
" in the worksheet '" & Me.Name & "' were modified on " & _
Format$(Now, "mm/dd/yyyy") & " at " & Format$(Now, "hh:mm:ss") & _
" by " & Environ$("username") & "."

With xMailItem
.To = "Email Address"
.Subject = "Worksheet modified - Update TI Survey Schedule Date"
.Body = xMailBody
.Display
End With
Set xRgSel = Nothing
Set xOutApp = Nothing
Set xMailItem = Nothing
End If
Application.DisplayAlerts = True
Application.ScreenUpdating = True
End Sub

 

[mumps]

What are the multiple columns and does the macro change in any way for each column?

 

[Sharonca]

Macro does not change in any for each column. Columns are:
AA, AC, AG, AI, BD, BF, BY, CG, CJ, DH, DJ, EB, ED

 

[mumps]

Try:

Private Sub Worksheet_Change(ByVal Target As Range)
    Dim xOutApp As Object, xMailItem As Object, xMailBody As String
    On Error Resume Next
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False
    If Intersect(Target, Range("AA:AA", "AC:AC", "AG:AG", "AI:AI", "BD:BD", "BF:BF", "BY:BY", "CG:CG", "CJ:CJ", "DH:DH", "DJ:DJ", "EB:EB", "ED:ED")) Is Nothing Then Exit Sub
    ActiveWorkbook.Save
    Set xOutApp = CreateObject("Outlook.Application")
    Set xMailItem = xOutApp.CreateItem(0)
    xMailBody = "Cell(s) " & Target.Address(False, False) & " in the worksheet '" & Me.Name & "' were modified on " & _
    Format$(Now, "mm/dd/yyyy") & " at " & Format$(Now, "hh:mm:ss") & " by " & Environ$("username") & "."
    With xMailItem
        .To = "Email Address"
        .Subject = "Worksheet modified - Update TI Survey Schedule Date"
        .Body = xMailBody
        .Display
    End With
    Application.DisplayAlerts = True
    Application.ScreenUpdating = True
End Sub

Using "Error Resume Next" can create problems because it disables all error messages so if your code errors out, you won't know which line of code is causing the error. It is always best to use error checking instead. Delete that line of code unless it is absolutely necessary for some reason. The same for "Application.DisplayAlerts = False". Usually this code is placed immediately before the line of code that would display the alert and "Application.DisplayAlerts = True" immediately after.

 

[Sharonca]

Try:

Private Sub Worksheet_Change(ByVal Target As Range)
    Dim xOutApp As Object, xMailItem As Object, xMailBody As String
    On Error Resume Next
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False
    If Intersect(Target, Range("AA:AA", "AC:AC", "AG:AG", "AI:AI", "BD:BD", "BF:BF", "BY:BY", "CG:CG", "CJ:CJ", "DH:DH", "DJ:DJ", "EB:EB", "ED:ED")) Is Nothing Then Exit Sub
    ActiveWorkbook.Save
    Set xOutApp = CreateObject("Outlook.Application")
    Set xMailItem = xOutApp.CreateItem(0)
    xMailBody = "Cell(s) " & Target.Address(False, False) & " in the worksheet '" & Me.Name & "' were modified on " & _
    Format$(Now, "mm/dd/yyyy") & " at " & Format$(Now, "hh:mm:ss") & " by " & Environ$("username") & "."
    With xMailItem
        .To = "Email Address"
        .Subject = "Worksheet modified - Update TI Survey Schedule Date"
        .Body = xMailBody
        .Display
    End With
    Application.DisplayAlerts = True
    Application.ScreenUpdating = True
End Sub

Using "Error Resume Next" can create problems because it disables all error messages so if your code errors out, you won't know which line of code is causing the error. It is always best to use error checking instead. Delete that line of code unless it is absolutely necessary for some reason. The same for "Application.DisplayAlerts = False". Usually this code is placed immediately before the line of code that would display the alert and "Application.DisplayAlerts = True" immediately after.

I received a Compile Error at the Range ("AA:AA", etc). Reason was wrong number of arguments or invalid Property assignment. Should these be listed as seperate lines?

 

[mumps]

Oops!!! Use this line to replace the old one:

If Intersect(Target, Range("AA:AA, AC:AC, AG:AG, AI:AI, BD:BD, BF:BF, BY:BY, CG:CG, CJ:CJ, DH:DH, DJ:DJ, EB:EB, ED:ED")) Is Nothing Then Exit Sub

 

[Sharonca]

That worked wonderfully and you are awesome. One more question. Each one of these status has a site # in column A. How would I include that corresponding site # in the subject line (preferably at the beginning of the subject line)

 

[mumps]

I'm not sure if I understood correctly. Try this line:

.Subject = Range("A" & Target.Row).Value & " Worksheet modified - Update TI Survey Schedule Date"

 

[Sharonca]

Also, how would I designate the value of "A" that should trigger the email. I think it goes in the line of above but not sure where or the syntax to use.

 

[Sharonca]

I'm not sure if I understood correctly. Try this line:

.Subject = Range("A" & Target.Row).Value & " Worksheet modified - Update TI Survey Schedule Date"

This worked PERFECTLY! Thank you