Hi All, I think we feel like both pros and cons because of these data file are located under the configurations. These are not actually config files and as Kasun said if these are under the config folder then anyone can feel to change and expect the changes in system after restarting. And same as Johan said, we can explain in the documentation clearly about the behaviour.
As an alternative solution, can't we have such files in some different root folder but not under the config or deployment ,because of these files are not either deployable artifact or configs. Then users will not misunderstand about these files and there can be its own behaviour as what we have now. *Harsha Thirimanna* *Associate Tech Lead | WSO2* Email: hars...@wso2.com Mob: +94715186770 Blog: http://harshathirimanna.blogspot.com/ Twitter: http://twitter.com/harshathirimann Linked-In: linked-in: http://www.linkedin.com/pub/harsha-thirimanna/10/ab8/122 <http://wso2.com/signature> On Wed, Jul 26, 2017 at 5:21 PM, KasunG Gajasinghe <kas...@wso2.com> wrote: > Hi Johann, > > > On Wed, Jul 26, 2017 at 3:22 PM, Farasath Ahamed <farasa...@wso2.com> > wrote: > >> Hi, >> >> email-admin-config.xml has the default email templates that are used for >> Email notifications. >> Since we have a UI to add/update new Email templates I don't think the >> user has to go to the registry and do any modifications. >> >> >> Thanks, >> Farasath >> >> Farasath Ahamed >> Software Engineer, WSO2 Inc.; http://wso2.com >> Mobile: +94777603866 >> Blog: blog.farazath.com >> Twitter: @farazath619 <https://twitter.com/farazath619> >> <http://wso2.com/signature> >> >> >> >> On Wed, Jul 26, 2017 at 1:30 PM, Johann Nallathamby <joh...@wso2.com> >> wrote: >> >>> Hi Kasun, >>> >>> I don't think we need to do this because, >>> >>> 1. This is the model we follow for some other files as well. >>> claim-config.xml >>> identity-event.properties >>> >>> > Yes, I think Isura mentioned the same. Let me revise my original query. > The problem I was trying to address is: > > 1. There is a config file under repository/conf/ which is only read during > start-up. Since it is read once, should it really be in the > repository/conf/? > > 2. If a config file is there under repository/conf/, then users expect any > changes to it will be visible to users. It is not that intuitive for a > first time user IMO. > > As the product is quite matured, I think it may be better to look into > these aspects to make the first time user experience better. Hence, the > request for $subject. > > > >> 2. This content goes not only to the super tenant but also to tenants as >>> default email templates. So if we change it in the middle tenants created >>> until then will have a different default template and the ones created >>> after that will have a different default template. >>> >>> > Yes, this is the current behavior, isn't it? > > Thanks for your feedback. > > >> I don't think we have any problems with current model. If we do this we >>> need to change our migration, etc. If the issue here is clarity for the >>> users it is better we explain it using better wordings in the docs. >>> >>> Regards, >>> Johann. >>> >>> ---------- Forwarded message ---------- >>> From: Kasun Gajasinghe (JIRA) <j...@wso2.org> >>> Date: Wed, Jul 19, 2017 at 1:54 AM >>> Subject: [jira] (IDENTITY-6167) Need to honor modifications to >>> email-admin-config.xml file >>> To: joh...@wso2.com >>> >>> >>> Kasun Gajasinghe >>> <https://wso2.org/jira/secure/ViewProfile.jspa?name=kasung%40wso2.com> >>> *created* an issue >>> >>> WSO2 Identity Server <https://wso2.org/jira/browse/IDENTITY> / [image: >>> Improvement] <https://wso2.org/jira/browse/IDENTITY-6167> IDENTITY-6167 >>> <https://wso2.org/jira/browse/IDENTITY-6167> >>> Need to honor modifications to email-admin-config.xml file >>> <https://wso2.org/jira/browse/IDENTITY-6167> >>> Issue Type: [image: Improvement] Improvement >>> Affects Versions: 5.3.0-GA >>> Assignee: Johann Nallathamby >>> <https://wso2.org/jira/secure/ViewProfile.jspa?name=johann%40wso2.com> >>> Created: 19/Jul/17 1:53 AM >>> Fix Versions: 5.4.0-GA >>> Priority: [image: High] High >>> Reporter: Kasun Gajasinghe >>> <https://wso2.org/jira/secure/ViewProfile.jspa?name=kasung%40wso2.com> >>> >>> At the moment, we save the email-admin-config.xml in the registry during >>> the very first server startup. After that, changes to the file are not >>> picked up. So, users are forced to go to the registry to update the config. >>> >>> This is a bit confusing since a user probably expects that the changes >>> to the file will be honored especially since it is located under >>> repository/conf folder. >>> >>> We can do some hashing mechanism and verify whether the contents in the >>> registry and the file matches. And, then we can update the registry >>> accordingly. >>> [image: Add Comment] >>> <https://wso2.org/jira/browse/IDENTITY-6167#add-comment> Add Comment >>> <https://wso2.org/jira/browse/IDENTITY-6167#add-comment> >>> >>> This message was sent by Atlassian JIRA (v7.2.2#72004-sha1:9d51328) >>> [image: Atlassian logo] >>> >>> >>> >>> -- >>> Thanks & Regards, >>> >>> *Johann Dilantha Nallathamby* >>> Senior Lead Solutions Engineer >>> WSO2, Inc. >>> lean.enterprise.middleware >>> >>> Mobile - *+94777776950* >>> Blog - *http://nallaa.wordpress.com <http://nallaa.wordpress.com>* >>> >> >> > > > -- > > *Kasun Gajasinghe*Associate Technical Lead, WSO2 Inc. > email: kasung AT spamfree wso2.com > linked-in: http://lk.linkedin.com/in/gajasinghe > blog: http://kasunbg.org > phone: +1 650-745-4499 <(650)%20745-4499>, 77 678 0813 > > > _______________________________________________ > Dev mailing list > Dev@wso2.org > http://wso2.org/cgi-bin/mailman/listinfo/dev > >
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
