Right now this code makes total sense, but we have and use a constant for _active_ accounts on the User model: `User::ACTIVE_STATUSES` Right now that only includes "active" but we've had others in the past and I think it would be best to either use the constant here or kill it all together until we need it again.