Skip to content

Commit

Permalink
Use remote_ip instead of ip (#466)
Browse files Browse the repository at this point in the history
Correct an issue where the location of users in Mixpanel continues to be wrong.
  • Loading branch information
millerti authored Feb 21, 2025
1 parent c2e00a6 commit bbe22b2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app/lib/generic_event_tracker.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ def self.for_request(request)
url_params = request.params.slice("client_agency_id", "locale")
defaults = {
# Not setting device_id because Mixpanel fixates on that as the distinct_id, which we do not want
ip: request.ip,
ip: request.remote_ip,
cbv_flow_id: request.session[:cbv_flow_id],
client_agency_id: url_params["client_agency_id"],
locale: url_params["locale"],
Expand Down

0 comments on commit bbe22b2

Please sign in to comment.