Customizing the web template is a good idea. (do we have to support html email ?) . Perso, I prefer to start with a configuration file in the file system (~/.continuum/continuum.xml or something else). I think about using that (in the module continuum-configuration) to fix CONTINUUM-1489 (the proxy configuration could be here).
-- Olivier 2008/6/13 Martin van den Bemt <[EMAIL PROTECTED]>: > Hi everyone, > > Before I start sending patches that get rejected, I thought it was best to > discuss this on the list > first. > The mail template are in continuum-core, which means that if you want to > customize continuum, you > need to start maintaining a separate patch / source tree for this > specifically or perform some > really nasty magic to override them. > > Currently in MailContinuumNotifier.java the location is constructed based on > these commands : > > String packageName = getClass().getPackage().getName().replace( '.', '/' ); > String templateName = packageName + "/templates/" + project.getExecutorId() + > "/" + source + ".vm"; > > The simplest solution is to first check if there is a custom template eg > String templateName = "custom/" + packageName + "/templates/" + > project.getExecutorId() + "/" + > source + ".vm"; > > if that exists use that, if not, use the default.. Maybe not the best > solution (like make it > configurable through the webinterface), but it is simple and low cost and > just makes it possible to > override without touching any continuum modules. > > Thoughts ? > > Mvgr, > Martin >
