dukex / mixpanel

Golang Mixpanel Client
MIT License
61 stars 33 forks source link

Add plumbing to allow autogeolocation on the mixpanel side #3

Closed tmpvar closed 6 years ago

tmpvar commented 6 years ago

In short, empty Event.IP/Update.IP results in a ip=1 to be appended to the url params for people updates and events. By sending ip=1, mixpanel will automatically geolocate using the ip of the requester during the API request.


It looks like this was the intended behavior, but unfortunately it did not work as described (https://github.com/dukex/mixpanel/blob/master/mixpanel.go#L39-L40)