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

Solve issues #177 and #178 #180

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

mentgit
Copy link

@mentgit mentgit commented Aug 25, 2016

This solves issue #177 and #178.

@mentgit mentgit changed the title Update actiontemplate.php Solve issues #177 and #178 Aug 25, 2016
@Klap-in
Copy link
Collaborator

Klap-in commented Aug 25, 2016

What does your new setting do? You do not override templates, but existing pages. Therefore I do not yet understand the setting name.

Please note that the setting text is still missing in the configuration manager.

ps: could you please describe the real change your are performing in a commit, instead of mentioning the file name. In the future that is more helpful if someone is reading back the changes that occur in that file. Thanks!

@mentgit
Copy link
Author

mentgit commented Aug 25, 2016

My idea was: A page will be overwritten with the content of the template. But this may be misleading. How is "pageoverwrite"? I also added the description of the option.

@mentgit
Copy link
Author

mentgit commented Aug 26, 2016

I renamed the option templateoverwrite to pageoverwrite. Now there is probably no ambiguity any more.

@erdnaxeli
Copy link

This actually does not solve #177 at all.

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.

3 participants