search

Ecwid / ecwid-mailchimp

MailChimp API Wrapper for Java
Apache License 2.0
86 stars 83 forks source link

Added temples support #8

Closed UberNick closed 11 years ago

UberNick commented 11 years ago

I implemented the templates api call

http://apidocs.mailchimp.com/api/rtfm/templates.func.php

basiliscus commented 11 years ago

Hi Nick,

Thank you for the contribution. Overall, the changes look good. My only concern is the TemplateInformation.id field: shouldn't it be declared as Integer instead of String?

UberNick commented 11 years ago

Thanks basiliscus.

You're right about the field type. It was working as a String but should be an Integer. I made the changes, tested locally, and checked in.

BTW, if you do merge, you're welcome to update the copyright comments to reference ecwid.

basiliscus commented 11 years ago

Thanks, I've merged the changes and released a new maven artifact (version 1.3.0.6). It will become available in Maven Central in few hours.