Code that emails the last saved version of the Activeworkbook is broken

[rhudgins]

I have the following code that is supposed to email the last saved version of the Activeworkbook. The code instead is emailing a blank workbook. Can someone help me fix this issue? I have attached a sample of my workbook for reference.

Please Login or Register  to view this content.

[Syrkrasi]

is the email server you're testing from blocking .xlsm files?
Because I've tested your email code and its working

[Logit]

.

Please Login or Register  to view this content.

[rhudgins]

Syrkrasi- No my email server is not blocking xlms files

Logit- Adding the file path you suggested ("C:\Users\My\Downloads\Live Trade Monitor - IB.xlsm") will not work because the Activeworkbook that is supposed to be sent is a newFile created everyday with a different name ("PnL m_dd_yy") that is created one step before in the macro Private Sub SaveDailyPnLFile. I have copied the SaveDailyPnLFile code below.

Please Login or Register  to view this content.

[rhudgins]

Can anyone help me with this?

[Logit]

.
If you can make certain the workbook you save is the only file to be in the folder ...

Have a line at the beginning of the macro that deletes all files in the folder before saving anything there.

Change your path of the attachment to select "all .xlsm files" ______*.xlsm______in the folder.
Since there will only be one file in the folder, that is the one that will be selected as the attachment.

[rhudgins]

That is not possible because I need a new file saved everyday PnL 8_21_17, PnL 8_22_17 ect.

My code has been running fine everyday for the last 5 years. A few weeks ago all of a sudden it stopped working.

The portion of the code ".Attachments.Add ActiveWorkbook.FullName" is attaching a blank workbook with the correct file name. If I go to the folder where the file is saved the workbook has the correct file name as well as data in the file.

Did excel/outlook make changes that would prevent my code from sending an attachment with data in the workbook?

[Logit]

.
From reading the Forums, some things written in pre-2016 Excel don't run well or not at all in Excel 2016.

If you need to retain the copies of the workbooks each day for future reference, move the saved file to a storage folder instead of deleting it at the beginning of the macro.

[rhudgins]

Thanks I assume it is a 2016 issue.

Do you know the best code to email the Activeworkbook in excel 2016?

[Logit]

.
I don't own Excel 2016 ... .but you can try this macro :

Please Login or Register  to view this content.

[rhudgins]

That code you provided is not working either. Thanks trying to help

[Logit]

.
Hmmm ......

Makes me wonder about whether I want to upgrade to Excel 16.

I am so sorry I haven't been able to assist you.

[Logit]

.
Since the other thread is closed ....

I hesitate to post a macro due to Excel 16 presenting issues for you. However, let's give it a go ?

Please Login or Register  to view this content.

[bakerman2]

Another one to test.

Please Login or Register  to view this content.

[rhudgins]

The second code that you provided worked great! Thanks for all of your help and patients!

[Logit]

.
Me ?

You are welcome.

[bakerman2]

@ Logit

Maybe me ?

@ rhudgins

You're welcome and thanks for rep+.