Refactor some User fields
We have a mess of inconsistent fields for User:
- locked
- deactivated
- is_moderator
- is_admin
- discoverable
- confirmation_pending
- approval_pending
- password_reset_pending
I think there are a few ways to address this. The only odd one is password_reset_pending
. I'm unsure if it really needs changing because it's a little different: it's an actionable requirement for the user, not a specific account attribute or status.
This with or without the is_ prefix would be an improvement
- is_locked
- is_active
- is_moderator
- is_admin
- is_discoverable
- is_confirmed
- is_approved
- force_password_reset (?)
Other suggestions welcome!
Edited by feld