You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Devise token auth overrides email_required? and email_changed? methods just to disable devise validations. Overriding email_required? seems ok as its a method defined by devise so people can expect devise token auth to override it. But overriding email_changed? is not the cleanest way to disable validations. I wanted to use it in condition for callbacks but ended up using email_change after coming to know that email_changed? is overridden by devise token auth
As devise suggests itself, that if there is a need to modify devise validations, then its better to just implement your own validators and exclude validatable module.
The text was updated successfully, but these errors were encountered:
Devise token auth overrides
email_required?
andemail_changed?
methods just to disable devise validations. Overridingemail_required?
seems ok as its a method defined by devise so people can expect devise token auth to override it. But overridingemail_changed?
is not the cleanest way to disable validations. I wanted to use it in condition for callbacks but ended up usingemail_change
after coming to know thatemail_changed?
is overridden by devise token authAs devise suggests itself, that if there is a need to modify devise validations, then its better to just implement your own validators and exclude validatable module.
The text was updated successfully, but these errors were encountered: