A regression was introduced in https://gerrit.wikimedia.org/r/#/c/311753/ where the dismiss event totalInteractionTime property is being sent as null. That's because logData is being reset (in the closePopup method) before it's sent as part of the dismiss event.
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Regression: Correctly log total interaction time for `dismiss` event | mediawiki/extensions/Popups | master | +10 -7 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | None | T145665 Verify interaction times are logged correctly with hovercards off and on (with the same actions) | |||
| Resolved | • Jhernandez | T146927 dismissed event total interaction time is not being reported correctly |
Event Timeline
Comment Actions
Change 313245 had a related patch set uploaded (by Bmansurov):
Regression: Correctly log total interaction time for dismiss event
Comment Actions
Change 313245 merged by jenkins-bot:
Regression: Correctly log total interaction time for dismiss event