| ppelberg | |
| Jul 19 2024, 10:05 PM |
| F54103990: CitoidError_ManualAdd.png | |
| Jul 19 2024, 10:05 PM |
T364595 introduced a new instance of the Add manually call to action:
This task involves the working of instrumentation to said affordance so that we can measure and understand who engages with it and how they engage with it.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Log when the add-manually button in the error message is pressed | mediawiki/extensions/Citoid | master | +7 -1 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Samwalton9-WMF | T362379 Several major news websites (NYT, Reuters...) block citoid | |||
| Resolved | ppelberg | T368988 Investigate Citoid feature use | |||
| Resolved | Ryasmeen | T370561 Add logging to Citoid's "Add manually" call to action |
Change #1057915 had a related patch set uploaded (by DLynch; author: DLynch):
[mediawiki/extensions/Citoid@master] Log when the add-manually button in the error message is pressed
Sequence will be:
| feature | action | meaning |
| citoid | window-open-from-command | Inspector opens |
| citoid | automatic-generate | "create" button clicked |
| CitoidInspector | automatic-generate-fail-searchResults | failure happened when getting the search results (added in T363292) |
| citoid | automatic-generate-fail | failure happened, error message shown (added in T363292) |
| citoid | automatic-generate-manual-fallback | "add manually" button clicked (added by this patch) |
| citoid | panel-switch | switching panels |
| citoid | panel-switch-manual | switching to the manual panel |
| citoid | dialog-manual-choose | clicked a button in the manual panel |
| reference | window-open-from-command | base cite reference-editor opens |
I didn't put it under automatic-generate-fail because WMDE seems to be using that prefix for logging error-reasons in T363292.
Patch also fixes a minor bug that would cause the panel-switch+panel-switch-manual events to fire twice when the "add manually" button was pressed.
@MNeisler, I'm assigning this ticket over to you to +1 or suggest any changes to the instrumentation @DLynch proposed in T370561#10023765 (see bolded bit).
The proposed new event (automatic-generate-manual-fallback) and the sequence of events noted in T370561#10023765 looks good to me. No changes.
@DLynch - Let me know once this has been deployed and I can do a quick QA check.
Change #1057915 merged by jenkins-bot:
[mediawiki/extensions/Citoid@master] Log when the add-manually button in the error message is pressed