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 ElasticEmail Service #24

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

MaizerGomes
Copy link
Contributor

@MaizerGomes MaizerGomes commented Jun 10, 2020

Added ElasticEmail as a new service.
Some work is still needed for opens and clicks tracking but at least the email sending is working.

To be able to use webhooks, must have a paid PRO Plan which I don't have and there's very little documentation available about the webhook. Only found this page..
Thats what I used as reference to decode the webhook payload.

I guess the only thing missing is the Feature Test.

@MaizerGomes MaizerGomes changed the title Add ElastiEmail Service Add ElasticEmail Service Jun 10, 2020
@joshuafranks joshuafranks marked this pull request as draft June 10, 2020 20:54
@joshuafranks joshuafranks requested a review from JonoB June 10, 2020 20:54
@MaizerGomes MaizerGomes marked this pull request as ready for review June 11, 2020 22:24
@JonoB JonoB added enhancement New feature or request help wanted Extra attention is needed labels Jun 15, 2020
@JonoB
Copy link
Contributor

JonoB commented Jun 15, 2020

Thanks for the PR!

I'd like to get feedback from a couple of people that are using this and can test it thoroughly.

@JonoB
Copy link
Contributor

JonoB commented Jul 31, 2020

Hey @MaizerGomes - thanks for sending this through PR. I think we'll have to wait for at least a couple of people to test this with Pro credentials.

I'll leave it open in the meantime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants