How to attach PDFs from desktop folder to various emails?

[lapot]

Hi all

I have a folder on desktop called "CreditPdf" which has some pdf files saved in it and this is variable.

I also have an excel sheet which has email addresses on Colum J which are unique to the ShortCodes on Column K.

I like to attach PDFs in "H:\Desktop\CreditPDF" folder to the emails under the Colum J, if the Invoice Name matches with the PDFs in the "CreditPDF" folder. If it is same code under Colum K, I like one email with multiple attachments?

is this possible?

I attach a sample sheet and picture

kind regards

Stay home be safe!!

[Keebellah]

So you already know how to send the emails, well once you have that then all you need is add attachment including the FULLPATH and Filename
in the email code

Please Login or Register  to view this content.

[lapot]

thanks for reply but i cant do multiple emails depending on different Client codes

[Keebellah]

if it's multiple attachments then you have to create an array with the fullpath and filename in each item of that array.
You have attached an xlsm file (which implies that it contains macros) but there are no macros.
So what do you have ?
And, how many attachments can it be? Varying from 1 to many?

[Keebellah]

Take a look here. helped me very much.
Enough samples and code samples
http://www.rondebruin.nl/win/section1.htm

[lapot]

Sorry I have now attached a macro on the sample which pulls all the PDFs into one email from the folder but I need PDFs attached to relevant Client Codes in separate emails.

[Keebellah]

I'll see and check tomorrow, past midnight now.

[Keebellah]

Okay, clear, I see what you've done, which is not really incorrect, only you need to select the file for each appropriate client.
My approach would be have two macros, one to create the pdf's which can be run if the pdf is not already present.
The other macro would be to process the email.
Just to make sure I understand: you have duplicate clients and you want to send just one mail with all the attachments for that client. Correct?
Or separate emails even if it's the same client per invoice?

[lapot]

I want to send just one mail with all the attachments for that client.Not separate email for each PDF. Is it possible?

Many thanks

[Keebellah]

Yes, of course that's possible only you have to rethink your approach.
This means that as you sample is now, it will be three emails and two of these have more than one attachment. Correct?

[lapot]

on this occasion yes but some times it can be 5 or 10 for same client code, it is variable.

[Keebellah]

Of course, but once you put it together it will work.
But, have you thought of what you want to do after the message(s) have been sent?
Like it is now the macro will resend the same message again when you run it again.
You have not built in a status like mail sent is true or false

[lapot]

i will deal with that later, at the moment it just displays the email and i will chose send or not send.this is not a big issuer for me,.

my main problem is i cant do the attachments for each client

[Keebellah]

Okay, will see what I can do to help you.

[Keebellah]

Well, you're code is IMHO a mess, and I don't feel like building it all up again in a logical way.
First the paths and so are not accessible to me so I have to improvise and set other paths, which is no problem, but I'm not in for it.
What I do want to say is that you can use your own code but you need to do it as follows:

I don't know why you have With oEmailItem embedded into each other.
I would make sure that you extract the unique list of Client names.
once you have that you process each name and the the whole of column B.
Inside that look you add the attachments with the client's invoice and when completed you can take the next one.
that means the whole of column B in this case get processed three times.
But, I noticed that the second occurrence or Radian is not equal to the one above, the second one has a space after the name, that is something to keep in mind when comparing strings, one with a space at the end is different from one with out it

Another thing you can reduce the follwoing string:

Please Login or Register  to view this content.

to

Please Login or Register  to view this content.

[lapot]

Hi mate

thank you for your effort, I know the code is extra things that it shouldn't be there , you can avoid them. Signature is not an issue you can take it out from the code too.

I need to create a filer on the Client code (Column K) each time it will bring up the word document links for that client (Column F) then created the PDF documents then save into the CreditPDF folder then attach to the email whatever in the folder then delete everting in the PDF folder.

Then filter next Client code, create all the PDF documents for the filtered client code, attach again all to the email and delete all the PDFs from the folder so on …..

does it make sense ?

thank you in advance

[Keebellah]

It all makes sense but it's very unmanageable.
You could consider like I said loop though a unique list, create the pdf(s) for that client and then mail, then process the next one.
You could even use the temp folder since you're deleting the pdfs after mailing.
Let me see if I can put the time in and at least give you the basic code so you can do your thing with your paths.

[lapot]

I really apricate mate if you could

[Keebellah]

I have not run the macro because I did not want to alter anything in the table.
Give it a run and see what it does.
So this is untested by me, just written out using part of your code and adding some of mine

PS modified the attachment, missed a thing in the code.

[lapot]

I get this errorAttachment 672483

[Keebellah]

The attachment you included is invalid.

[Keebellah]

What's the exact error message?

[lapot]

please see the attachment

thank you

[Keebellah]

Typo- how about typing the correct syntax.
I said I could not test it.
.Attachments
I expected a little more initiative

[lapot]

Sorry it was my bad, it has been a long day, I must have overlooked the spelling. I corrected it but it did 3 times same email. I was watching inside the folder ,it wasn't putting just the relevant document inside the folder, it was putting all 5 document on each email with same subject and body.

here is the screen shot

[Keebellah]

Like I said , it was just the code I wrote no testing possible without changing the content, I’ll pick it up later today and see what I can come up with to recreate your data
Pity, troubleshooting takes time

[Keebellah]

Pls check if the file you’re using is the last one I updated it later the same day adding spme lined of code between the for each ci and the for each ring Clearing the str variables

[Keebellah]

This is the part that I mean:

Please Login or Register  to view this content.

I also see that the Hi ... refers to the wrong column, but that YOU can change too.
Like I said, will see if I can modify the sample file with some fake word files and see what I can do.

[Keebellah]

Macro code needed several corrections due to lack of testing not detected:

See screenshot of the result
Paste this code replacing the macro in your file (just this one macro

Please Login or Register  to view this content.

[Keebellah]

The body text formatting needs to be done correctly but that's a question of inserting the correct HTML tags in the strBody textstring

[Keebellah]

I placed the modified vba code in the sample with your file settings. Should work too.
Also did something to the HTML text formatting and looks better

[lapot]

Wow, what can I say !! you are a legend, if I was near you I would send you a bottle of drink. This is amazing. It works perfectly.. I cant thank you enough..

[Keebellah]

Glad to have been able to help and to hear it works as planned.
Don't forget to mark the post as solved.

Cheers,

Happy coding