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

Feat: No email to confirmed leads #6

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

KreativeKrise
Copy link
Contributor

Since my website uses a lot of lead magnets and people always sign up for the same segment, I need a feature to not send emails to already confirmed contacts. (See #5)

For this I took the idea from the FAQ (Maybe we should only send the email, when the contact does not have the tags, that we would add in the success case?) but made it a bit more flexible.

To stop sending the emails, you need to enable the option in the plugin settings and configure the form action accordingly.

You can select for each form in the action configuration whether the tags, segments or a field of the contact should be compared with the success values from the action. There is now also the possibility to select a field for the status of the doi and what the value must be in case of a success. If the contact is recognized as confirmed, the email will not be sent, but the configured actions except logging and removing from DNC will be executed anyway.

Attention: The plugin can now be enabled and disabled in the plugin settings. By default, the plugin is disabled, which is why the DOI plugin will not be executed after an update.

Disclaimer: I am not a developer and created the feature with trial and error.

@sebastian-fahrenkrog
Copy link
Member

Hey @KreativeKrise !
Thank you so much for your input! I was not getting any notifications from github and just seeing your pull request!

I will check it out and test it and then merge it!

Thank you!

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.

2 participants