Page MenuHomePhabricator

Deprecate and remove mw.eventLog.submitInteraction()
Closed, ResolvedPublic2 Estimated Story Points

Description

Description

mw.eventLog.submitInteraction is currently being used for a few instruments. As we are doing with other functions like mw.eventLog.submitClick and mw.eventLog.newInstrument we would have to deprecate and remove this one as part of T408059: [GOAL] Tidy up EventLogging. mw.testKitchen.getInstrument() exists since a while ago as the replacement also for this case.

Acceptance Criteria

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
JVanderhoop-WMF moved this task from READY TO GROOM to Backlog on the Test Kitchen board.
phuedx updated the task description. (Show Details)
phuedx added a subscriber: cjming.

@cjming effectively did this as part of T415370: Migrate all instruments using mw.eventLog.submitInteraction to mw.testKitchen.getInstrument(). As I mentioned in the Experiment Platform team's Slack channel:

There is only one reference of mw.eventLog.submitInteraction() in all codebases indexed by Codesearch and that is in a comment by @cjming noting that it's being decommissioned. There are two on-wiki references and those are archived Metrics Platform documentation.

It's Done™