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

Add optional Fields CC, BCC and ContentType to informationcollector class #69

Open
fwoldt opened this issue Aug 14, 2020 · 6 comments
Open

Comments

@fwoldt
Copy link

fwoldt commented Aug 14, 2020

It would be good to have CC, BCC + contenttype (text/html or text/plain) as an option for the informationcollection content classes. So the infocollectionmails can be controlled by the configuration of the contentobject in the backend.

Optional CC & BCC email field are very usefull.

contentype text/html or text/plain is sometimes used for technical mails - a form is send to an other technical system as plain/text version

We have already a patch for the infocollection bundle 1.9 i attached as a tar.gz.

vendor/netgen/information-collection-bundle/bundle/Mailer/Mailer.php
vendor/netgen/information-collection-bundle/bundle/Factory/AutoResponderDataFactory.php
vendor/netgen/information-collection-bundle/bundle/Factory/EmailDataFactory.php
vendor/netgen/information-collection-bundle/bundle/DependencyInjection/ConfigurationConstants.php
vendor/netgen/information-collection-bundle/bundle/DependencyInjection/Configuration.php
vendor/netgen/information-collection-bundle/bundle/Value/EmailData.php
vendor/netgen/information-collection-bundle/bundle/Constants.php

information-collection-bundle_1.9_cc_bcc_contentype_patch.tar.gz

hopefully you can add these functionality to your bundle

I see you are refactoring your extension at the moment - maybe you could add these function, too.

Thank's

@MarioBlazek
Copy link
Contributor

@fwoldt thanks for your suggestions. This was the problem we encountered a long time ago, so I have already covered this in v2. The plan is to release v2 by the end of the month.

@MarioBlazek
Copy link
Contributor

I will take a look and try to squeeze this into v1 also.

@fwoldt
Copy link
Author

fwoldt commented Aug 14, 2020

great news :-)

@fwoldt
Copy link
Author

fwoldt commented Mar 5, 2021

What is about this enhancement?
This Enhancement is really usefull for us. Please merge it so i do not patch it again and again into our projects.
Thank's

@fwoldt
Copy link
Author

fwoldt commented Feb 20, 2023

any update?

@emodric
Copy link
Member

emodric commented Feb 20, 2023

Hi @fwoldt, this is currently out of scope. The old version of Info Collector (1.x) is deprecated and does not receive any more features.

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

No branches or pull requests

3 participants