Skip to content

Createsend#9

Open
plaftmp wants to merge 18 commits into
seevibes:masterfrom
plaftmp:createsend
Open

Createsend#9
plaftmp wants to merge 18 commits into
seevibes:masterfrom
plaftmp:createsend

Conversation

@plaftmp

@plaftmp plaftmp commented Aug 10, 2016

Copy link
Copy Markdown
Contributor

Same email and password to connect to Campaign Monitor as for zendesk

Omniauth gem to use : gem 'omniauth-createsend' (github)

No environment variable are needed

For it to work as it, you'll need to modify gems/oauth2-1.2.0/lib/oauth2/client.rb

should look like:

l.138 response = request(options[:token_method], token_url, opts)
l.139 error = Error.new(response)

Need to be change to:

l.138 opts[:body][:redirect_uri] = "http://localhost:3006/auth/createsend/callback"
l.139 response = request(options[:token_method], token_url, opts)
l.140 error = Error.new(response)

Comment thread lib/connectors/createsend.rb Outdated

def account_identifiers
{
#there is nothing giving by omniauth that we could use

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

giving by omniauth ==> from Omniauth

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