django-unique-user-email

django-unique-user-email copied to clipboard

Add URLs (and maybe templates?)

2

It's easy enough to define a `urls.py` so folks can just include that once. Maybe templates too: * All the `contrib.auth` views have the `extra_context` attribute ([e.g.](https://docs.djangoproject.com/en/dev/topics/auth/default/#django.contrib.auth.views.LoginView.extra_context)) allowing a base...

carltongibson

Fix #7 required email constraint

4

fix #7

trottomv

User.email field is not mandatory and remain `blank=True`

10

Hi @carltongibson, I'm here again, because your solution seems really interesting to me. I noticed, though, that the `email` field remains with `blank=True` parameter and since it's not a required...

trottomv

Write up the WHY (was "Hey")

2

Hey Carlton, this just showed up in my feed. Just wanted to let you know that I made a reusable lib for this, because I was quiet frustrated with the...

tfranzel

The package makes no sense otherwise. I can **say** all I like, but better to fail loudly.

carltongibson

Test Cases and Tutorial

1

There are three levels that should prevent duplicate emails: 1. `ModelForm`. 2. `Model.full_clean()` 3. `Model.save()` (via constraint). ## Tutorial * Show that you get 1 & 2 without the migration....

carltongibson

Debug print in form clean() method

1

It feels to me like a forgotten debug print, otherwise I find it difficult to understand why there should be a print in `clean`. https://github.com/carltongibson/django-unique-user-email/blob/a383fa85ed7fa3002202e26c6027b7fb4b665b5f/src/unique_user_email/forms.py#L55

washeck

drworm

Shouldn't the unique constraint be case-insensitive?

3

Email addresses look like this: username@hostname . The hostname part is case-insensitive. The username part is case-sensitive in theory, but case-insensitive in practise. Shouldn't the unique constraint on the email...

Flimm