Adds site settings view

bookwyrm/forms.py

@@ -195,7 +195,14 @@ class ShelfForm(CustomForm):
         model = models.Shelf
         fields = ['user', 'name', 'privacy']
 
+
 class GoalForm(CustomForm):
     class Meta:
         model = models.AnnualGoal
         fields = ['user', 'year', 'goal', 'privacy']
+
+
+class SiteForm(CustomForm):
+    class Meta:
+        model = models.SiteSettings
+        exclude = []

bookwyrm/templates/layout.html

@@ -92,7 +92,7 @@
                     </li>
                     {% if perms.bookwyrm.create_invites %}
                     <li>
-                        <a href="/invite" class="navbar-item">
+                        <a href="{% url 'settings-invites' %}" class="navbar-item">
                             Invites
                         </a>
                     </li>

bookwyrm/templates/settings/admin_layout.html

@@ -0,0 +1,42 @@
+{% extends 'layout.html' %}
+{% block content %}
+
+<header class="block column is-offset-one-quarter pl-1">
+    <h1 class="title">{% block header %}{% endblock %}</h1>
+</header>
+
+<div class="block columns">
+    <nav class="menu column is-one-quarter">
+        {% if perms.bookwyrm.create_invites %}
+        <h2 class="menu-label">Manage Users</h2>
+        <ul class="menu-list">
+            <li>
+                {% url 'settings-invites' as url %}
+                <a href="{{ url }}"{% if url in request.path %} class="is-active" aria-selected="true"{% endif %}>Invites</a>
+            </li>
+        </ul>
+        {% endif %}
+        {% if perms.bookwyrm.edit_instance_settings %}
+        <h2 class="menu-label">Instance Settings</h2>
+        <ul class="menu-list">
+            <li>
+                {% url 'settings-site' as url %}
+                <a href="{{ url }}"{% if url in request.path %} class="is-active" aria-selected="true"{% endif %}>Site Configuration</a>
+                {% if url in request.path %}
+                <ul class="emnu-list">
+                    <li><a href="{{ url }}#instance-info">Instance Info</a></li>
+                    <li><a href="{{ url }}#images">Images</a></li>
+                    <li><a href="{{ url }}#footer">Footer Content</a></li>
+                    <li><a href="{{ url }}#registration">Registration</a></li>
+                </ul>
+                {% endif %}
+            </li>
+        </ul>
+        {% endif %}
+    </nav>
+    <div class="column content">
+        {% block panel %}{% endblock %}
+    </div>
+</div>
+
+{% endblock %}

bookwyrm/templates/settings/manage_invites.html

@@ -1,8 +1,8 @@
-{% extends 'layout.html' %}
+{% extends 'settings/admin_layout.html' %}
+{% block header %}Invites{% endblock %}
 {% load humanize %}
-{% block content %}
+{% block panel %}
-<div class="block">
+<section class="block">
-    <h1 class="title">Invites</h1>
     <table class="table is-striped">
         <tr>
             <th>Link</th>
@@ -22,12 +22,12 @@
         </tr>
         {% endfor %}
     </table>
-</div>
+</section>
 
-<div class="block">
+<section class="block">
     <h2 class="title is-4">Generate New Invite</h2>
 
-    <form name="invite" action="/invite/" method="post">
+    <form name="invite" action="{% url 'settings-invites' %}" method="post">
         {% csrf_token %}
         <div class="field is-grouped">
             <div class="control">
@@ -46,5 +46,5 @@
 
         <button class="button is-primary" type="submit">Create Invite</button>
     </form>
-</div>
+</section>
 {% endblock %}

bookwyrm/templates/settings/site.html

@@ -0,0 +1,84 @@
+{% extends 'settings/admin_layout.html' %}
+{% block header %}Site Configuration{% endblock %}
+
+{% block panel %}
+
+<form action="{% url 'settings-site' %}" method="POST" class="content">
+    {% csrf_token %}
+    <section class="block" id="instance-info">
+        <h2 class="title is-4">Instance Info</h2>
+        <div class="control">
+            <label class="label" for="id_name">Instance Name:</label>
+            {{ site_form.name }}
+        </div>
+        <div class="control">
+            <label class="label" for="id_instance_tagline">Tagline:</label>
+            {{ site_form.instance_tagline }}
+        </div>
+        <div class="control">
+            <label class="label" for="id_instance_description">Instance description:</label>
+            {{ site_form.instance_description }}
+        </div>
+        <div class="control">
+            <label class="label" for="id_code_of_conduct">Code of conduct:</label>
+            {{ site_form.code_of_conduct }}
+        </div>
+    </section>
+
+    <hr aria-hidden="true">
+
+    <section class="block" id="images">
+        <h2 class="title is-4">Images</h2>
+        <div class="field is-grouped">
+            <div class="control">
+                <label class="label" for="id_logo">Logo:</label>
+                {{ site_form.logo }}
+            </div>
+            <div class="control">
+                <label class="label" for="id_logo_small">Logo small:</label>
+                {{ site_form.logo_small }}
+            </div>
+            <div class="control">
+                <label class="label" for="id_favicon">Favicon:</label>
+                {{ site_form.favicon }}
+            </div>
+        </div>
+    </section>
+
+    <hr aria-hidden="true">
+
+    <section class="block" id="footer">
+        <h2 class="title is-4">Footer Content</h2>
+        <div class="control">
+            <label class="label" for="id_support_link">Support link:</label>
+            <input type="text" name="support_link" maxlength="255" class="input" id="id_support_link" placeholder="https://www.patreon.com/bookwyrm" value="{{ site.support_link }}">
+        </div>
+        <div class="control">
+            <label class="label" for="id_support_title">Support title:</label>
+            <input type="text" name="support_title" maxlength="100" class="input" id="id_support_title" placeholder="Patreon" value="{{ site.support_title }}">
+        </div>
+        <div class="control">
+            <label class="label" for="id_admin_email">Admin email:</label>
+            {{ site_form.admin_email }}
+        </div>
+    </section>
+
+    <hr aria-hidden="true">
+
+    <section class="block" id="registration">
+        <h2 class="title is-4">Registration</h2>
+        <div class="control">
+            <label class="label" for="id_allow_registration">Allow registration:
+            {{ site_form.allow_registration }}
+        </div>
+        <div class="control">
+            <label class="label" for="id_registration_closed_text">Registration closed text:</label>
+            {{ site_form.registration_closed_text }}
+        </div>
+    </section>
+
+    <footer class="block">
+        <button class="button is-primary" type="submit">Save Changes</button>
+    </footer>
+</form>
+{% endblock %}

bookwyrm/tests/views/test_invite.py

@@ -44,5 +44,5 @@ class InviteViews(TestCase):
         request.user.is_superuser = True
         result = view(request)
         self.assertIsInstance(result, TemplateResponse)
-        self.assertEqual(result.template_name, 'manage_invites.html')
+        self.assertEqual(result.template_name, 'admin/manage_invites.html')
         self.assertEqual(result.status_code, 200)

bookwyrm/urls.py

@@ -49,8 +49,12 @@ urlpatterns = [
     re_path(r'^password-reset/(?P<code>[A-Za-z0-9]+)/?$',
             views.PasswordReset.as_view()),
 
+    # admin
+    re_path(r'^settings/site-settings',
+            views.Site.as_view(), name='settings-site'),
     # invites
-    re_path(r'^invite/?$', views.ManageInvites.as_view()),
+    re_path(r'^settings/invites/?$',
+            views.ManageInvites.as_view(), name='settings-invites'),
     re_path(r'^invite/(?P<code>[A-Za-z0-9]+)/?$', views.Invite.as_view()),
 
     # landing pages

bookwyrm/views/__init__.py

@@ -24,6 +24,7 @@ from .search import Search
 from .shelf import Shelf
 from .shelf import user_shelves_page, create_shelf, delete_shelf
 from .shelf import shelve, unshelve
+from .site import Site
 from .status import CreateStatus, DeleteStatus
 from .updates import Updates
 from .user import User, EditUser, Followers, Following

bookwyrm/views/invite.py

@@ -24,7 +24,7 @@ class ManageInvites(View):
                 user=request.user).order_by('-created_date'),
             'form': forms.CreateInviteForm(),
         }
-        return TemplateResponse(request, 'manage_invites.html', data)
+        return TemplateResponse(request, 'settings/manage_invites.html', data)
 
     def post(self, request):
         ''' creates an invite database entry '''
@@ -36,7 +36,7 @@ class ManageInvites(View):
         invite.user = request.user
         invite.save()
 
-        return redirect('/invite')
+        return redirect('/settings/invites')
 
 
 class Invite(View):

bookwyrm/views/site.py

@@ -0,0 +1,31 @@
+''' manage site settings '''
+from django.contrib.auth.decorators import login_required, permission_required
+from django.shortcuts import redirect
+from django.template.response import TemplateResponse
+from django.utils.decorators import method_decorator
+from django.views import View
+
+from bookwyrm import forms, models
+
+
+# pylint: disable= no-self-use
+@method_decorator(login_required, name='dispatch')
+@method_decorator(
+    permission_required(
+        'bookwyrm.edit_instance_settings', raise_exception=True),
+    name='dispatch')
+class Site(View):
+    ''' manage things like the instance name '''
+    def get(self, request):
+        ''' edit form '''
+        site = models.SiteSettings.objects.get()
+        data = {
+            'title': 'Site Settings',
+            'site_form': forms.SiteForm(instance=site)
+        }
+        return TemplateResponse(request, 'settings/site.html', data)
+
+    def post(self, request):
+        ''' edit the site settings '''
+
+        return redirect('/settings/site-settings')