fortscan Posted July 18, 2013 Report Share Posted July 18, 2013 This is critical for many companies who don't want to fill their customer's inbox space with a PDF for every invoice notification. Even when optimized and without embedding fonts, the emails end up with an average weight of 70-100k. * It can overload the SMTP queue when sending large amount of notifications, * It forces the customer to actually download an attachment * It may even put the relaying SMTP IPs in danger of being blacklisted or SMTP-filtered because of large number of identical attachment parameters (i.e. 2k emails in 1h, all of them with a 100k attachment) * In some cases it can interfere with privacy policies in the sense that customers are receiving sensitive/confidential information over an unsecure channel like email, instead of having to first authenticate in the billing system over SSL to be able to access such information A simpler approach can be more useful: let the customer know that an invoice has been generated (plain text email), and IF they want to look at the PDF, ask them to login to the billing system. If this is not a current feature, could you at least suggest a workaround, even if it implies hacking/modifying part of the includes or the email generation class ? Thanks. Quote Link to comment Share on other sites More sharing options...
Cody Posted July 18, 2013 Report Share Posted July 18, 2013 Good suggestion! You'll have to get your hands a little dirty, but you can disable sending attachments in invoice delivery emails by commenting out lines 80 - 85 in /components/invoice_delivery/invoice_delivery.php $attachments = array( /* array( 'path' => $inv_path, 'name' => $attachment_name, 'encoding' => "base64", 'type' => $this->company_settings['inv_mimetype'] ) */ ); Quote Link to comment Share on other sites More sharing options...
fortscan Posted July 18, 2013 Author Report Share Posted July 18, 2013 Thanks a lot for your quick reply! We really appreciate it. I'll try this today with a couple of demo customers. Quote Link to comment Share on other sites More sharing options...
Paul Posted July 18, 2013 Report Share Posted July 18, 2013 This has been assigned to CORE-589 and is scheduled for version 3.1 currently. It will likely be a setting for the company and client group. fortscan, MemoryX2 and Michael 3 Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.