Merge pull request #195 from open-plan-tool/feature/add-account-email…

…-verification-process

Feature/add account email verification process

app/users/forms.py

@@ -1,7 +1,5 @@
 from django import forms
 from django.contrib.auth.forms import UserCreationForm, UserChangeForm
-from crispy_forms.helper import FormHelper
-from crispy_forms.layout import Layout, Field
 
 from .models import CustomUser
 

app/users/models.py

@@ -1,5 +1,4 @@
 from django.contrib.auth.models import AbstractUser
-from django.db import models
 
 
 class CustomUser(AbstractUser):

app/users/views.py

@@ -9,14 +9,11 @@
 from django.core.mail import EmailMessage, send_mail, BadHeaderError
 from django.db.models import Q
 from django.http import HttpResponse
-from django.http.response import HttpResponseRedirect
 from django.shortcuts import render, redirect
 from django.template.loader import render_to_string
-from django.urls.base import reverse
 from django.utils.encoding import force_bytes
 from django.utils.http import urlsafe_base64_encode, urlsafe_base64_decode
 from django.views.decorators.http import require_http_methods
-
 from .forms import CustomUserCreationForm, CustomUserChangeForm
 from .models import CustomUser
 
@@ -28,11 +25,9 @@
 def signup(request):
     if request.method == "POST":
         form = CustomUserCreationForm(request.POST)
-        # print(form.errors.as_data())
         if form.is_valid():
             user = form.save(commit=False)
-            # user.is_active = False
-            user.is_active = True
+            user.is_active = False
             user.save()
             current_site = get_current_site(request)
             mail_subject = "Activate your account."
@@ -47,9 +42,9 @@ def signup(request):
             )
             to_email = form.cleaned_data.get("email")
             email = EmailMessage(mail_subject, message, to=[to_email])
-            # email.send()
-            # return HttpResponse('Please confirm your email address to complete the registration')
-            return HttpResponseRedirect(reverse("login"))
+            email.send()
+            messages.info(request, "Please confirm your email address to complete the registration")
+            return redirect("home")
     else:
         form = CustomUserCreationForm()
     return render(request, "registration/signup.html", {"form": form})
@@ -64,11 +59,11 @@ def activate(request, uidb64, token):
     if user is not None and default_token_generator.check_token(user, token):
         user.is_active = True
         user.save()
-        return HttpResponse(
-            "Thank you for your email confirmation. Now you can login your account."
-        )
+        messages.success(request, "Thank you for your email confirmation. Now you can login your account.")
+        return redirect("login")
     else:
         return HttpResponse("Activation link is invalid!")
+        return redirect("home")
 
 
 @login_required