HomePhabricator

eventLogging: Add missing perceivedWait property

Authored by phuedx on May 3 2017, 9:14 AM.

Description

eventLogging: Add missing perceivedWait property

Per the Popups schema [0], the "dismissed" action expects the
perceivedWait property to be set.

Happily, the time until the preview was shown is already recorded and
used to determine whether or not to create a "dismissed" or
"dwelledButAbandoned" event.

Reducer changes:

  • When creating the "dismissed" event, set the perceivedWait property to the already accumulated timeToPreviewShow.

[0] https://meta.wikimedia.org/wiki/Schema:Popups

Bug: T164256
Change-Id: I3fd253f1c2ff5fa8e24c9225060d728ffd8dfd27

Details