existing email

[jamesgeddes]

added form validation to reject existing emails. If email exists, returns "You've already registered! Please login"

Future improvements
Could easily be updated to redirect user to login once the login page exists.

[jamesgeddes]

@CarwynNelson sorry for the radio silence over the past few weeks. I'll pick back up on the login form so would be grateful of your thoughts on this. only addition is the following on forms.py

def clean_email(self): if Member.objects.filter(email=self.cleaned_data['email']).exists(): raise forms.ValidationError("You've already registered! Please login") return self.cleaned_data['email']

[jamesgeddes]

Just to be clear, this resolves #34

[CarwynNelson]

Looks good. Could we add in a basic automated test for this? I believe the existing form tests should be an easy place to do it.

Something along the lines of:

1. Create a member with an email
2. try and register with that email
3. assert the form error was returned/thrown

I believe there are tests that do all of the above so you should be able to piece it together. If not just give me a shout :)

[jamesgeddes]

Have not had a look at the automated testing. Will have a go and let you know.