Fix for Issue #59 for location tracking based on $ip parameter

[etagwerker]

Hey @zevarito!

These commits solve the location tracking issue described in Issue #59.

It's a proposed implementation, with no additional tests. I just want to see what you think about it.

I've tested it locally and in production and it works.

It allows you to do this:

tracker = Mixpanel::Tracker.new MIXPANEL_API_KEY
tracker.set("12345", {:first_name => "John", :last_name => "Doe" }, { "$ip".to_sym => "186.137.140.183" } )

Thoughts?

I'd appreciate any comments on the code and potential tests.

[zevarito]

Hi @etagwerker!

Thanks for submitting your code.

Please take a look at my comment here: https://github.com/zevarito/mixpanel/issues/59#issuecomment-11640333

• Would be great to support old IP resolution as it is in: https://github.com/zevarito/mixpanel/blob/master/lib/mixpanel/event.rb#L30
• Tests passing and for new features are needed.
• Update Readme with new API.

Thanks!

[zevarito]

62 Was merged, please see comments on #59. Thanks.

[etagwerker]

@zevarito cool!