Can't get API to work at all about active_campaign

[jmburges]

same!

[shenst1]

same

[jasonfb]

@lorenzsell @jmburges @shenst1 -- I think the problem here is that master branch is not consistent with what is pushed to Rubygems.

see last known published gem https://github.com/mhenrixon/active_campaign/tree/v0.1.16

sync_contact became contact_sync
add_contact became contact_add
etc

[jasonfb]

this seems like all developer confusion to me. I went down these same rabbit holes, explained here: #84

@lorenzsell @jmburges @shenst1

as well, you must take note that the comment in the setup has the critical piece of hidden information: /admin/api.php must be added to your active campaign subdomain for it to work (this is all working on the V1 API)

client = ::ActiveCampaign::Client.new(
api_endpoint: 'https://[YOUR ACTIVE CAMPAIGN DOMAIN]/admin/api.php', # e.g. 'https://yourendpoint.api-us1.com/admin/api.php'
api_key: '[YOUR-API-KEY]')

@mhenrixon -- as a point of feedback, your code has lots of comments with important hidden details, which other developers (like me and the ones above) won't read when we're moving quickly. I'd recommend making some of this stuff more explicit in the actual code so that our editors show it as a color we should pay attention to (and not grey, signaling irrelevance.) just to make it easier to grok for people new to the gem.

[jasonfb]

also to debug the actual API, I had to do a little low-level monkey patching. Since the published version of the gem is currently old and I see that master is refactored, I won't bother making a pull request. As well, I see that on the new gem code there are already begin/rescue to handle this.

however, when working on v0.1.16, you will get crashes on client.rb line 71 if the JSON fails to parse.

To debug it, I had to re-write the method like to see the actual responses from the API:

I would highly recommend patterns like this when making wrappers because you can't anticipate all of the things that can (will) go wrong with these responses and it's probably best just to leave the responses spit back to us the developer to figure out where went wrong.

 def request(method, api_method, data)
      req = create_request method, api_method, data
      response = HTTPI.send(method, req)
      begin
        response = JSON.parse(response.body)
      rescue StandardError => e
        puts "*********  There was an error parsing the response from ActiveCampaign: #{response.inspect}"
        raise(e)
      end

      normalize(response)
    end