Merge pull request #40 from OpenTechFund/feature/107-fix-none-page-bug-in-template

Make the author field required

opentech/public/news/migrations/0004_make_author_required.py

+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.8 on 2018-01-15 12:59
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('news', '0003_newspageauthor'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='newspageauthor',
+            name='author',
+            field=models.ForeignKey(default=1, on_delete=django.db.models.deletion.PROTECT, related_name='+', to='wagtailcore.Page'),
+            preserve_default=False,
+        ),
+    ]

opentech/public/news/models.py

@@ -59,9 +59,7 @@ class NewsPageAuthor(Orderable):
     )
     author = models.ForeignKey(
         'wagtailcore.Page',
-        null=True,
-        blank=True,
-        on_delete=models.SET_NULL,
+        on_delete=models.PROTECT,
         related_name='+',
     )