Show errors in edit user form and remove email

email was triggering the form to reject becuase of uniqueness

bookwyrm/forms.py

@@ -99,7 +99,7 @@ class EditUserForm(CustomForm):
     class Meta:
         model = models.User
         fields = [
-            'avatar', 'name', 'email', 'summary', 'manually_approves_followers'
+            'avatar', 'name', 'summary', 'manually_approves_followers'
         ]
         help_texts = {f: None for f in fields}
 

bookwyrm/templates/edit_user.html

@@ -2,24 +2,32 @@
 {% block content %}
 <div class="block columns">
     <div class="column is-half">
+        <h2 class="title">Profile</h2>
+        {% if form.non_field_errors %}
+        <p class="notification is-danger">{{ form.non_field_errors }}</p>
+        {% endif %}
         <form name="edit-profile" action="/edit_profile/" method="post" enctype="multipart/form-data">
             {% csrf_token %}
-            <h2 class="title">Profile</h2>
             <p class="block">
                 <label class="label" for="id_avatar">Avatar:</label>
                 {{ form.avatar }}
+                {% for error in form.avatar.errors %}
+                <p class="help is-danger">{{ error | escape }}</p>
+                {% endfor %}
             </p>
             <p class="block">
                 <label class="label" for="id_name">Display name:</label>
                 {{ form.name }}
+                {% for error in form.name.errors %}
+                <p class="help is-danger">{{ error | escape }}</p>
+                {% endfor %}
             </p>
             <p class="block">
                 <label class="label" for="id_summary">Summary:</label>
                 {{ form.summary }}
-            </p>
+                {% for error in form.summary.errors %}
-            <p class="block">
+                <p class="help is-danger">{{ error | escape }}</p>
-                <label class="label" for="id_email">Email address:</label>
+                {% endfor %}
-                {{ form.email }}
             </p>
             <p class="block">
                 <label class="checkbox label" for="id_manually_approve_followers">

bookwyrm/view_actions.py

@@ -173,7 +173,11 @@ def edit_profile(request):
 
     form = forms.EditUserForm(request.POST, request.FILES)
     if not form.is_valid():
-        return redirect('/')
+        data = {
+            'form': form,
+            'user': request.user,
+        }
+        return TemplateResponse(request, 'edit_user.html', data)
 
     request.user.name = form.data['name']
     if 'avatar' in form.files: