Page MenuHomePhabricator

[PHP] Remove dispatch method
Closed, ResolvedPublic2 Estimated Story Points

Description

Description

Remove the dispatch() method from the PHP client library.

The dispatch() method was initially introduced during early iterations of the Metrics Platform that used the monoschema. Now that we've pivoted to using the data contract model, this and related methods should be removed in favor the the api submit methods.

This task is dependent on making sure no instruments are currently using monoschema to send events.

Acceptance Criteria

  • The dispatch() and supporting methods are removed from the PHP client.
  • Update relevant tests.
  • Documentation is updated accordingly.

Event Timeline

phuedx set the point value for this task to 2.Jan 16 2024, 11:51 AM
phuedx subscribed.

AFAICT this isn't blocked on anything. Being bold and bringing this into the current sprint.

Change 1009480 had a related patch set uploaded (by Phuedx; author: Phuedx):

[mediawiki/extensions/EventLogging@master] Use wikimedia/metrics-platform 4.0.0

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

Change 1009484 had a related patch set uploaded (by Phuedx; author: Phuedx):

[mediawiki/vendor@master] Update wikimedia/metrics-platform to 4.0.0

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

Change #1009480 merged by jenkins-bot:

[mediawiki/extensions/EventLogging@master] Use wikimedia/metrics-platform 4.1.0

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

Change #1009484 merged by jenkins-bot:

[mediawiki/vendor@master] Update wikimedia/metrics-platform to 4.1.0

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