-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Rename Template into TemplateProcessor #216
Comments
I basically agree that
|
Creating directory just for one class is not a good idea, on my mind, at all. :) |
I'm not 100% sure but doe this use the Mail Merge functionality? If so it might make sense to use |
The process worked like mail merge, but we don't use mail merge component. Which reminded me that we don't have mail merge component, yet. |
Suggest renaming
Template
class intoTemplateProcessor
, because latter matches its purpose the best.Template is not an instance of the
Template
class, it's a file on hard drive. The class loads template's content and processes it. Please, take a look at the definition also.TemplateProcessor
class could evolve into a separate component in the future, but, for now, I suggest just renaming.The text was updated successfully, but these errors were encountered: