Query about pending reviewables reminder


Should the following:

discourse/pending_reviewables_reminder.rb at 3602f83cf443d9736e94747f25b31a8e9a318e93 · discourse/discourse (github.com)

… not sort by ASC instead? Would make more sense to tag recently active moderators rather than moderators that may not have been seen in >1 month - or even more.


Wouldn’t descending do that?

01/01/2022 01/04/2022
01/02/2022 01/03/2022
01/03/2022 01/02/2022
01/04/2022 01/01/2022

You’re right. This seems to be an edge-case with our forum because there are some “dummy” accounts used to mask identities using a plugin, which have a last_seen_at as null because they were bulk created users.

1 Like

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.