Code review comment for lp:~therp-nl/server-env-tools/7.0-add_email_template_conditional_attachment

Thanks for the comments. Setting to work in progress as I'm currently thinking about whether or not to reset the m2m relationship when attachments are copied as this currently leads to duplicate attachments on the template (which is useful when you are configuring multiple attachments on the same template but rather disturbing when you are not.