Ori.livneh has submitted this change and it was merged.

Change subject: On AddNewAccount, log server-side using logEvent()
......................................................................


On AddNewAccount, log server-side using logEvent()

---- Patch 1 : looking for feedback ----
Works on localhost except for null cookies; are these the right
elements?

---- Patch 2 : separate version key-val pair ----

---- Patch 3 : DarTar comments ----
Set creator_userid to -1 for anonymous user creating her account;
drop clicktracking-session; fix cookie retrieval; version=1

---- Patch 4 : Ori comments ----
Underscores; explicit self_made; better comments.

---- Patch 5 : no user token ----
Comment out mediaWiki.user.id cookie logging.

Change-Id: I10d5354682e4865c5140d08a3e43a5c5b858911a
---
M Experiments.hooks.php
1 file changed, 38 insertions(+), 0 deletions(-)

Approvals:
  Ori.livneh: Verified; Looks good to me, approved


--
To view, visit https://gerrit.wikimedia.org/r/25214
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I10d5354682e4865c5140d08a3e43a5c5b858911a
Gerrit-PatchSet: 5
Gerrit-Project: mediawiki/extensions/E3Experiments
Gerrit-Branch: master
Gerrit-Owner: Spage <[email protected]>
Gerrit-Reviewer: DarTar <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: Spage <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to