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 option to use European sparkpost servers #70

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add option to use European sparkpost servers #70

wants to merge 1 commit into from

Conversation

olivierh65
Copy link

Add an option to use domain eu.sparkpost.com (european acounts).
Add also an option to allow Sparkpost tracking. Usefull to track transactional mails.

Add an option to use domain eu.sparkpost.com (european acounts).
Add also an option to allow Sparkpost tracking. Usefull to track transactional mails.
@simonjohnparker
Copy link

Hi, when do you think this PR will get merged as we need to use the EU Sparkpost servers? Thanks.

@nganivet
Copy link
Contributor

@simonjohnparker Not on our priority list for now, specifically as we need some small tweaks to the submitted PR and it has a conflict with the latest version of the extension.

@petergerner
Copy link

see my rebased patch for #70

@herbdool
Copy link
Contributor

@nganivet we have a client that requires this too. Could help with getting some version of the PR merged in.

@herbdool
Copy link
Contributor

@petergerner I think the link above is supposed to be #78

@nganivet
Copy link
Contributor

@herbdool Can you switch the Yes/No to a drop-down for endpoint selection? The Yes/No is confusing and does not lend itself to more servers added in the future. A Select control, with the options taken from a constant variable declared at the top of the class would be clearer and more extensible. Thanks.

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.

5 participants