search

danesparza / MailChimp.NET

:envelope: .NET Wrapper for the MailChimp v2.0 API
MIT License
179 stars 119 forks source link

Modified 'merges' property on GetMemberInfo result #102

Closed slegay closed 10 years ago

slegay commented 10 years ago

The 'MemberMergeInfo' property of class 'MemberInfo' was incorrect and wasn't pulling any of the merge variables. This patch introduces the MemberInfoMergeData struct which uses custom JSON serialization to extract dynamic merge vars into a dictionary, and 'groupings' in its own strongly typed collection.

danesparza commented 10 years ago

So in the README, there is a sample of using a class inherited from MergeVar when subscribing a user. If somebody were to use that implementation -- it looks like they would not be using that class they created when getting data with MemberMergeInfo now -- is that correct?

Are you suggesting that the old method of creating a class inherited from MergeVar didn't work at all, it was broken somehow, or it just didn't work for you?