+1 Swapnil Thanks & Regards,
On Thu, Aug 17, 2017 at 2:42 PM, Marek Mosiewicz <marekmosiew...@gmail.com> wrote: > 17.08.2017 10:56 "Aditya Sharma" <aditya.sha...@hotwaxsystems.com> > napisał(a): > > > +1 > > > > Thanks and Regards, > > > > *Aditya Sharma* | Enterprise Software Engineer > > HotWax Systems <http://www.hotwaxsystems.com/> > > <https://www.linkedin.com/in/aditya-sharma-78291810a/> > > > > On Thu, Aug 17, 2017 at 10:39 AM, Arun Patidar < > > arun.pati...@hotwaxsystems.com> wrote: > > > > > +1 Swapnil for content driven template. > > > > > > Also, like idea to remove redundancy of email templates settings. > > > > > > -- > > > Thanks & Regards > > > --- > > > Arun Patidar > > > Manager, Enterprise Software Development > > > > > > > > > HotWax Systems Pvt Ltd.www.hotwaxsystems.com > > > > > > > > > On Thu, Aug 17, 2017 at 10:19 AM, Swapnil Mane < > > > swapnil.m...@hotwaxsystems.com> wrote: > > > > > > > Thanks Nicolas for your inputs and sharing more details. This > proposed > > > > model is making sense to me. > > > > Please give me some more time to look into the details, will get back > > to > > > > you in next week. > > > > > > > > Also, please see my comments inline. > > > > > > > > On Tue, Aug 15, 2017 at 1:52 AM, Nicolas Malin < > > nicolas.ma...@nereide.fr > > > > > > > > wrote: > > > > > > > > > Hello Swapnil, in line > > > > > > > > > > > > > > > Le 14/08/2017 à 04:35, Swapnil Mane a écrit : > > > > > > > > > >> Thank you Nicolas for your inputs and interest. I highly > appreciate > > > it. > > > > >> > > > > >> Based on my understanding, please see my comments inline and let > me > > > know > > > > >> if > > > > >> you have further inputs. > > > > >> > > > > >> On Fri, Aug 11, 2017 at 3:10 PM, Nicolas Malin < > > > > nicolas.ma...@nereide.fr> > > > > >> wrote: > > > > >> > > > > >> Hello Swapnil, > > > > >>> > > > > >>> In past I tried to refactoring email interface with the idea to : > > > > >>> * deprecate current ProductStoreEmailSetting to link it to > > > > >>> TemplateEmailSetting. The purpose is to centralize all email > > > > >>> configuration > > > > >>> in this entity > > > > >>> > > > > >>> We may have multiple product store and can have different email > > > > templates > > > > >> for them, ProductStoreEmailSetting will allow us to do that. > > > > >> > > > > > My fault, I'm not clear. ProductStoreEmailSetting and > > > > EmailTemplateSetting > > > > > are redundancy, > > > > > I'm in favor to keep all email template information in > > > > > EmailTemplateSetting and use ProductStoreEmailTemplate to link a > > email > > > > > template to a productStore throw a purpose. > > > > > So we can deprecate all email template fields in > > > ProductStoreEmailSetting > > > > > to centralize all this part in EmailTemplateSetting > > > > > > > > > > > > > +1 > > > > > > > > > > > > > * link TemplateEmailSetting with Content through > > > > >>> TemplateEmailSettingContent and TemplateEmailSettingContentType. > > > This > > > > >>> offert the possibilty to link header, body, footer or some more > > > complex > > > > >>> case like link documents, pdf invoice, order, etc ... > > > > >>> > > > > >>> Having content model with us, the customizable header, footer > > > > >> (decoratorContentId at content level) and other complex case can > be > > > > >> handled > > > > >> easily with content model. > > > > >> > > > > > Completely, except for attached file. I agree for rendering the > email > > > > > content, but if you want link the file to your email its more > easier > > to > > > > > indicate it on EmailTemplateSetting. > > > > > > > > > > An example, when you send a order confirmation, you want attach to > > this > > > > > email the the legal notice. We would be link directly the contentId > > > where > > > > > is the legal notice and an other content for the email body. > > > > > > > > > > > > > I guess, this can be achieved by ContentAssoc model, but yes, your > > > proposal > > > > of using TemplateEmailSettingContent and > TemplateEmailSettingContentTyp > > e > > > > is > > > > also looks reasonable to me. > > > > > > > > > > > > > * review all send email function to manage the content rendering > > > > >>> Yes, during the proposed implementation, we were planning to do > > this > > > as > > > > >>> well. > > > > >>> > > > > >>> But the time has been missed :( > > > > >>> If you are motivate, we can try to revive this idea ? > > > > >>> > > > > >> > > > > >> :-) > > > > >> I would love to hear more about your idea, will it be possible for > > you > > > > to > > > > >> share more information about this. > > > > >> > > > > > The issue where I started https://issues.apache.org/ > > > > jira/browse/OFBIZ-4333 > > > > > > > > > > Nicolas > > > > > > > > > > > > > > > > > > > > > - Best Regards, > > > > Swapnil M Mane, > > > > www.hotwaxsystems.com > > > > www.hotwax.co > > > > > > > > > > -- Renuka Srishti