Don't grant badges to users who don't exist?

Looking at one our badge pages I see

which got me curious - what happened to all those avatars?

Turns out these users don’t exist.

We probably shouldn’t grant badges to users that no longer exist? Or perhaps prevent them from being displayed? Regardless, they are ousting the avatars of the real users - the ones we care about!

8 Likes

Hmmm… something is not right here cause:

The grant is already joining into the user, it has to pull the user id from somewhere.

I wonder if the way this is scoped means that dependent destroy is not working right.

Regardless we can add a weekly cleanup.

Given when these badges were granted (at time of writing they were the most recent) I suspect they are granted when the user account doesn’t exist, as opposed to granted when the account exists and then not deleted on account deletion.

Will have to check.

1 Like

It is odd cause the rows have user ids, if the user is purged there is no where to pull these ids from.

1 Like