Page MenuHomePhabricator

QuickSurveys: Add sessionId to Schema:QuickSurveysResponses
Closed, ResolvedPublic3 Estimated Story Points

Description

Having a QuickSurvey is awesome.
Collecting answers is awesome.

But it would be more awesome if the Analytics team can relate these answers to the data being collected elsewhere.

What is not sessionId ?
It is not the user name or any private information.

What is sessionId ?
The sessionId is a random id generated in some location, to mark that some events belong to the same user. It allows single events to be grouped in order to represent an actual user experience.

What is the difference?
The goal is not to identify a user, the goal is to relate an answer to other events, anonymously.

Event Timeline

JGirault claimed this task.
JGirault raised the priority of this task from to Needs Triage.
JGirault updated the task description. (Show Details)
JGirault subscribed.

Change 254596 had a related patch set uploaded (by JGirault):
Adds an optional sessionId param to the QuickSurveysResponses schema, and adds an JavaScript API to dynamically set this sessionID (note that it can only be set dynamically after the initial configuration array as it is unique).

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

If the patch is approved, adding the sessionId to the data being sent by QuickSurveys would be as simple as adding this to the searchSatisfaction code:

if ( mw.extQuickSurveys ) {
	mw.extQuickSurveys.lib.sessionId.set( 'search-satisfaction', searchSessionId );
}
Jdlrobson triaged this task as Medium priority.Dec 3 2015, 5:58 PM
Jdlrobson subscribed.
debt renamed this task from Add sessionId to Schema:QuickSurveysResponses to QuickSurveys: Add sessionId to Schema:QuickSurveysResponses.Jan 28 2016, 4:49 PM

closing this out - the work has already been done.

Change 254596 abandoned by JGirault:
Add optional sessionId param to QuickSurveysResponses

Reason:
We don't need this patch anymore, it's been implemented differently.

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