Page MenuHomePhabricator

Include performer_is_temp contextual attribute with account_created events
Open, MediumPublic

Description

In useAccountCreationInstrument.js, account_created events should include performer_is_temp contextual attribute which is needed by:

metrics in GrowthBook. Right now the experiment owner must remember to include that contextual attribute when configuring the experiment in TK UI or GrowthBook.

It would be easier/simpler and better to collect this per-event (rather than experiment-wide) via:

- experiment.send( 'account_created' );
+ experiment.send( 'account_created', {}, [ 'performer_is_temp' ] );

And then the experiment owner doesn't have to select performer_is_temp when configuring the experiment, and there would be no risk of an experiment owner forgetting to do that step.

Event Timeline

DMburugu subscribed.

@Michael Would be worth considering in one of the upcoming sprints since we're still working on the temp accounts funnel hypothesis

Change #1318780 had a related patch set uploaded (by Bearloga; author: Bearloga):

[mediawiki/extensions/WikimediaEvents@master] Standardize reusable experiment instrumentation

https://gerrit.wikimedia.org/r/1318780

Noodling on some instrumentation updates and standardization. Will add Michael as reviewer after Sam S. has taken a look and feel good about it.

mpopov triaged this task as Medium priority.

Change #1318780 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@master] Standardize reusable instrumentation and tidy up

https://gerrit.wikimedia.org/r/1318780