Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

We should fall back to the default templates if we don't find a particular template in the custom template directory #6583

Closed
richvdh opened this issue Dec 20, 2019 · 1 comment · Fixed by #8037
Labels
z-p2 (Deprecated Label)

Comments

@richvdh
Copy link
Member

richvdh commented Dec 20, 2019

Currently, if you want to change a single template, you have to copy all of the templates to a custom location, and set email.template_dir. That's a nuisance, and it means that whenever we introduce a new template (as per Synapse 1.4.0), people have to make sure they manually install the new template.

It also repeatedly confuses people who think that they need to set template_dir and do so without realising the implications.

(Also: hiding the template_dir setting under email when it is used for non-email things is very silly.)

A better approach would be to look in the template_dir, and fall back to the default templates if the one we're looking for is missing.

@anoadragon453
Copy link
Member

anoadragon453 commented Aug 4, 2020

Hrm, apparently we already claim in the sample config that we do this:

If not set, default templates from within the Synapse package will be used.

but we don't, whoops.

Edit: Ah, this is saying that the default directory will be used, rather than the default template files will be used if a template is not present in the custom directory.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
z-p2 (Deprecated Label)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants