@JKatzWMF and @Jdlrobson will chat about his today

@Jdlrobson - obviously we did not get a chance to chat on Monday.

Given what the apps team implemented, I think we should do something similar:

event is triggered on page load 'shown'
event is triggered when 'read more' appears onscreen 'click', position of clicked item
'source' of suggestion (name of service used, or manual)--this way we can compare any improvements to suggestions.

ideally we could ensure links-clicked were accretive (rather than cannabalistic) by looking at overall clicks on the page (or at pageview #s referred to by the page), but not sure how that would be done.

Have added schema and clarified acceptance criteria for whoever is implementing this.

Are you working on this task, @Jdlrobson? It's assigned to you but it's in the todo column.

Change 249798 had a related patch set uploaded (by Bmansurov):
Add event logging to read more

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

Moving back to 'todo' as the blocked by task needs to be worked on first.

Change 251001 had a related patch set uploaded (by Bmansurov):
Add event logging to read more

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

Change 249798 abandoned by Bmansurov:
Add event logging to read more

Reason:
switched to the dev branch

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

There is a minor PHP code style violation that needs to be fixed.

251001 has a minor from @Krinkle. There's no associated C:-1, however, but I think his comment is worth following up on.

Ignore that last remark. I was looking at an older revision of the change.

I've reviewed 251001 and tested it locally and it LGTM.

There's a little bit more work around softening the dependency on EventLogging: the build is failing because of an unmet dependency on an EL module (hard) but we're using mw.track log events if EL is available (soft).

Change 251001 merged by jenkins-bot:
Add event logging to read more

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

I've added the following to the reading-web-staging RelatedArticles config:

$wgRelatedArticlesShowReadMore = true;
$wgRelatedArticlesLoggingSamplingRate = 1;

I've enabled the eventlogging extension, however I don't see that events are being sent. It maybe because mw.config.get('wgRelatedArticlesShowReadMore') is returning null. The second config variable is working though. I'd love to explore more, but since the patch is written by me, I'd rather someone else sign off on this task.

I'd love to explore more, but since the patch is written by me, I'd rather someone else sign off on this task.

Absolutely! I'm on it.

In T114303#1800230, @bmansurov wrote:

I've enabled the eventlogging extension, however I don't see that events are being sent. It maybe because mw.config.get('wgRelatedArticlesShowReadMore') is returning null.

The RelatedArticlesShowReadMore config variable isn't forwarded to the client like the others. Instead, it's used to control whether or not the bootstrap module gets added to the output.

I've tested this on reading-web-staging and can confirm that I see the ready, seen, and clicked RelatedArticles events being logged.