Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issues regarding templates #107

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Fix issues regarding templates #107

wants to merge 2 commits into from

Conversation

bertoost
Copy link
Contributor

  1. Changing the load custom template method to use Craft's frontend template path. Extends and includes could (and should) be relative to the default templates folder. This is how Craft is working by default too.
  2. Secondly changed the new confirmation email template setting. From the previous version, you could have set up a notification email template, which now becomes the only mail to the admin-user. The mail to the client becomes default again.
    When the new confirmation email template setting is not set, use the notification email template instead. Now backwards compatible.

Bert Oost added 2 commits October 21, 2016 14:05
…plate path relative to craft/templates/ folder.
…otification template. Fallback to default (backwards compatible)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant