Signup events for analytics
Summary
Push signup
events to analyze user conversions
- Closes #1240 (closed)
- Closes #1359 (closed)
ElasticSearch Mappings
- NEW: minds-metrics-* ->
bool pro_referrer
Steps to test
- Port forward sandbox sites elasticsearch cluster to your local machine (optional)
- Open your elasticsearch browser and open
minds-metrics-*
index - Sort by
@timestamp
descending - Open the review site, and create a new user. Don't do anything else after that to avoid unnnecesary events.
- Refresh your
minds-metrics-*
view on ElasticSearch browser - Ensure there's a
signup
metric with auser_guid
and acookie_id
value set (might be the third or fourth row from the top). Filtering by cookie_id (which has the same value a mwa cookie on the website). - Log out from the site.
- Create a new user using this link: https://goal-signup-cookie-id.minds.io/register?referrer=0xEMI
- Refresh your
minds-metrics-*
view on ElasticSearch browser - Ensure there's a new
signup
metric with referrer_guid and pro_referrer set totrue
.
Note: This one might be tricky for @xander-miller to test as it's a backend-only storage change. The main concern is to ensure new user registration works OK.
Estimated Regression Scope
New block of code executed at register/complete
event, might affect user sign-up.
added scoped labels
changed the description
mentioned in issue #1240 (closed)
assigned to @benhayward.ben
- Developer
@Xander if you need help getting set-up to test this one also, DM me.
approved this merge request
changed target branch from
master
toproduction
added 21 commits
-
dcaa9f5d...fec60070 - 4 commits from branch
production
- de6fd8ed - (wip): Frontend rendering module
- 7e2c2364 - Merge branch 'master' of https://gitlab.com/Minds/engine into epic/SSR
- dcaea0d4 - (chore): ci fixes
- f2751134 - Merge branch 'master' of https://gitlab.com/Minds/engine into epic/SSR
- 9a498785 - (chore): ci for preprod
- 5562dca5 - (chore): export pro on api config
- 1d9730a6 - Merge branch 'master' of https://gitlab.com/Minds/engine into epic/SSR
- 1ccf1085 - (chore): allow master to push to preprod too
- d3b9a8ec - Merge branch 'epic/SSR'
- e4fdcb9e - (chore): january token gift
- 550c417c - Merge branch 'production' of https://gitlab.com/Minds/engine
- 85275399 - (chore): bring sandbox helm chart branch to master
- 2d4ca7d6 - (feat): backend for captcha - front#646 (closed)
- 7c362781 - Merge branch 'feat/646-captcha' into 'master'
- 5c3e5b05 - Add environment support to Feature services
- 5e9b7bb9 - Merge branch 'goal/unleash-sync-namespaces' into 'master'
- f880bb3a - Merge remote-tracking branch 'origin/master' into goal/signup-cookie-id
Toggle commit list-
dcaa9f5d...fec60070 - 4 commits from branch
changed target branch from
production
tomaster
- Developer
@benhayward.ben I just re-started review site for this MR.
approved this merge request
- Developer
@benhayward.ben You couldn't create a new account because I needed to create a matching
front
branch that had the captcha fixed. - DeveloperResolved by Emiliano Balbuena
Give me a ping when this is ready Emi - I think you said you needed to add email confirmation?
- Last reply by Emiliano Balbuena
resolved all threads
approved this merge request
added scoped label and automatically removed label
assigned to @xander-miller and unassigned @benhayward.ben
- Developer
- Resolved by Emiliano Balbuena
added scoped label and automatically removed label
unapproved this merge request
changed the description
added 11 commits
-
f880bb3a...5c9218c9 - 10 commits from branch
master
- bbb4eb82 - Merge remote-tracking branch 'origin/master' into goal/signup-cookie-id
-
f880bb3a...5c9218c9 - 10 commits from branch
added 1 commit
- 257133da - (feat): Referrer Pro status and missing fields
resolved all threads
added scoped label and automatically removed label
assigned to @benhayward.ben and unassigned @xander-miller
changed the description
- Developer
assigned to @brianhatchet, @xander-miller, and @markeharding and unassigned @edgebal
- Developer
user Registration works. Ben's screen shots of the analytics results are satisfactory proof for me that that part is tested and working.
approved this merge request
approved this merge request
merged
mentioned in commit c28af3ce