Skip to content

Commit ed21537

Browse files
committed
Track member id and username in project user events
Added member id and member user name to properties of requesting/approving project membership. This change enables us to compute conversion rate between invited users and users that actually join project.
1 parent 44e53c1 commit ed21537

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

lib/code_corps/analytics/segment_traits_builder.ex

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,12 @@ defmodule CodeCorps.Analytics.SegmentTraitsBuilder do
7878
}
7979
end
8080
defp traits(%CodeCorps.ProjectUser{} = record) do
81-
record = record |> Repo.preload(:project)
81+
record = record |> Repo.preload([:project, :user])
8282
%{
8383
project: record.project.title,
84-
project_id: record.project_id
84+
project_id: record.project_id,
85+
member_id: record.user.id,
86+
member: record.user.username
8587
}
8688
end
8789
defp traits(%CodeCorps.StripeConnectAccount{} = account) do

test/lib/code_corps_web/controllers/project_user_controller_test.exs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,9 @@ defmodule CodeCorpsWeb.ProjectUserControllerTest do
5454

5555
tracking_properties = %{
5656
project: project.title,
57-
project_id: project.id
57+
project_id: project.id,
58+
member: user.username,
59+
member_id: user.id
5860
}
5961

6062
assert_received {:track, ^user_id, "Requested Project Membership", ^tracking_properties}
@@ -106,7 +108,9 @@ defmodule CodeCorpsWeb.ProjectUserControllerTest do
106108

107109
tracking_properties = %{
108110
project: project.title,
109-
project_id: project.id
111+
project_id: project.id,
112+
member: record.user.username,
113+
member_id: record.user.id
110114
}
111115

112116
assert_received {:track, ^user_id, "Approved Project Membership", ^tracking_properties}

0 commit comments

Comments
 (0)