Scott Wheeler commented on Upcoming change in API limit calculations

Here's a slightly updated version of our snippet -- I realized looking again at the old one that it'd eat non-429 connection errors rather than rethrowing them.  This fixes that:

module ActiveResource
  class Connection
    def request_with_sleeper(*args, &block)
      if self === ShopifyAPI::Base.connection
        if ShopifyAPI::Base.connection.response && ShopifyAPI.credit_maxed?
          sleep(0.5)
        end
        begin
          request_without_sleeper(*args, &block)
        rescue ActiveResource::ConnectionError => ex
          if ex.response.code.to_s == '429'
            sleep(0.5)
            retry 
          else
            raise ex
          end
        end
      else
        request_without_sleeper(*args, &block)
      end
    end

    alias_method_chain :request, :sleeper
  end
end