Help with Object Required error

[reberryjr]

I'm at a complete loss here. I'm trying to add a value to a range of cells...something I've done countless times. This time, I'm getting an Object required error. The only nuance I can think of is the range, but the row numbers are being identified correctly.

Sub ImportReqResults()

Application.DisplayAlerts = False
Application.ScreenUpdating = False

Dim m As Workbook, s As Workbook
Dim mD As Worksheet, sD As Worksheet
Dim mDLR As Long, sDLR As Long, mDNLR As Long

Set m = ThisWorkbook
Set mD = m.Sheets("Data")

mDLR = mD.Range("C" & Rows.Count).End(xlUp).Row
mDNR = mD.Range("C" & Rows.Count).End(xlUp).Row + 1

'I perform various updates here

mDNLR = mD.Range("C" & Rows.Count).End(xlUp).Row


'Adds the Data Source name in the Data Source column of this workbook.
With mD.Range("A" & mDNR) & ":" & mD.Range("A" & mDNLR)
    .Value = "Req Results" 'Error appears here.
End With

'Adds Today's date in the Source Ingested column of this workbook.
With mD.Range("B" & mDNR) & ":" & mD.Range("B" & mDNLR)
    .Value = "=Today()"
    .NumberFormat = "MM/DD/YY"
    .Value = .Value
End With

Application.ScreenUpdating = True
Application.DisplayAlerts = True

End Sub

 

[RoryA]

You've got the syntax wrong - it should be:

With mD.Range("A" & mDNR & ":A" & mDNLR)

rather than:

With mD.Range("A" & mDNR) & ":" & mD.Range("A" & mDNLR)

and the same for the B column range below that.

 

[reberryjr]

You've got the syntax wrong - it should be:

With mD.Range("A" & mDNR & ":"A" & mDNLR)

rather than:

With mD.Range("A" & mDNR) & ":" & mD.Range("A" & mDNLR)

and the same for the B column range below that.

Thanks @RoryA . I had to make a small tweak, and wound up with this working snippet.

With mD.Range("A" & mDNR & ":" & "A" & mDNLR)

 

[RoryA]

Oops - I left an extra quote in. I've amended my previous answer - there is no need for ":" & "A" when you can just use ":A"

 

[reberryjr]

Oops - I left an extra quote in. I've amended my previous answer - there is no need for ":" & "A" when you can just use ":A"

That makes sense.

 

[Joe4]

That makes sense.

In the future, when marking a post as the solution, please mark the actual post that contains the solution, not your post acknowledging that another post is the solution.
I have updated this thread for you.