django-allauth icon indicating copy to clipboard operation
django-allauth copied to clipboard

email_changed signal should be sent when changing a primary email using EmailAddress.change

Open prakhar1144 opened this issue 3 years ago • 2 comments

When changing an email using EmailAddress.change() https://github.com/pennersr/django-allauth/blob/c50a30acc0e4553297b0ddb697d55a290ed6d038/allauth/account/models.py#L65

There should be a condition as follows: If the email being changed is a primary email then email_changed signal should be sent. As of now it doesn't happens.

prakhar1144 avatar Jul 07 '21 05:07 prakhar1144

That change() method is actually dead code and not used by allauth at al. It should be removed altogether.

pennersr avatar Dec 10 '21 14:12 pennersr

I think add_email should be used for any use-case stemming from your need to use change method @prakhar1144

arthtyagi avatar Sep 05 '22 15:09 arthtyagi

The change() method is no longer present.

pennersr avatar Aug 09 '23 21:08 pennersr