Does that sound ok to you?

I was chatting in RTL that this may make more sense to implement as a follow on to the pinia store refactor in T422607: Define and add Pinia storage for PersonalDashboard Review Changes module.

This is very minor, but it looks like there's now more spacing between the Review Changes header and the Header Message:

Hi @MNeisler, question for you now that we've turned on enwiki with a 10% sampling rate.
How will this impact retention rate measurements? For example, if we're only sampling 10% but miss a second visit (if the user wasn't sampled during the second visit) then our retention rates will incorrectly look like they are lower?
Not sure if you have already addressed this or not, but just curious what you recommend?

Dashboard here.

Verified this is fixed on patch demo, but would love others to test on various phones to make sure there are no regressions.

In T419251#11772562, @Samwalton9-WMF wrote:

Just to clarify - we are able to link this specifically to the local Automoderator username/account?

When I wrote that log message proposal I was looking at some other messages that weren't user-specific, e.g.:

I see that almost every other log is "User [did something]" though, so I think I'm fine with the way you've done it here!

Ok, sounds fine to me, thanks for catching that though!

In T419251#11772459, @Samwalton9-WMF wrote:

Ah - does that apply to the log message too (where we also wanted to include a link to the CC form)?

In T419251#11772243, @Samwalton9-WMF wrote:

Today I Learned each log has its own description! Building on your suggestion with a little more specificity, since we can be wordy here:

This is a log of edits that Automoderator would have reverted, given its configuration at the time of the log entry.

And link Automoderator to the CC form. If this is wiki markup then I guess that would be [[Special:CommunityConfiguration/AutoModerator|Automoderator]] or [[Special:CommunityConfiguration/AutomoderatorMultilingual|Automoderator]] as appropriate.

What would we like the log description message to be?
Maybe something like:
"This is a log of edits that AutoModerator would have reverted."

Stalling this while we figure out an automated testing strategy for the team.

Verified that this is working as expected on idwiki.

Resolving as @Samwalton9-WMF is now able to view the dashboard without crashing.

In T419251#11748539, @Samwalton9-WMF wrote:

Huh, interesting. Creating that redirect on each wiki might be a pain. I'm happy to drop the 'learn more' part of the summary.

It does not appear that external links are allowed to be rendered in log lines, should we update the link to be a local one that then redirects to mediawiki https://www.mediawiki.org/wiki/Moderator_Tools/Automoderator?

What do we want the log action and action type to be?
See here for an example: https://www.mediawiki.org/wiki/Manual:Log_actions.

Is the logstash error we're seeing, indicating that the query for retrieving reviewed edit count is timing out.

$revisionCount = $this->dbr
			->newSelectQueryBuilder()
			->field( '1' )
			->table( 'revision' )
			->join( 'change_tag', null, 'ct_rev_id = rev_id' )
			->join( 'change_tag_def', null, 'ctd_id = ct_tag_id' )
			->where( [
				'rev_actor' => $actorId,
				'ctd_name' => [ 'mw-undo', 'mw-reverted', 'mw-manual-revert' ]
			] )
			->limit( 1000 )
			->caller( __METHOD__ )
			->fetchRowCount();