add ability to stop replying to individuals

 - closes #34

api/serializers.py

@@ -131,6 +131,7 @@ class Meta:
             'full_name',
             'is_archived',
             'is_blocking',
+            'do_not_reply',
             'last_sms',
         )
 

apostello/admin.py

@@ -42,7 +42,11 @@ class KeywordAdmin(admin.ModelAdmin):
 @admin.register(models.Recipient)
 class RecipientAdmin(admin.ModelAdmin):
     """Admin class for apostello.models.Recipient."""
-    list_display = ('full_name', 'number', 'is_blocking', 'is_archived', )
+    list_display = ('full_name',
+                    'number',
+                    'is_blocking',
+                    'is_archived',
+                    'do_not_reply', )
 
 
 @admin.register(models.RecipientGroup)

apostello/assets/js/components/contact_row.jsx

@@ -5,10 +5,12 @@ const ContactRow = (props) => {
   const className = props.is_blocking ? 'warning' : '';
   const lastSms = props.contact.last_sms === null ?
     { content: '', time_received: '' } : props.contact.last_sms;
+  const doNotReplyElem = <div className="ui horizontal red label">No reply</div>;
+  const doNotReply = props.contact.do_not_reply ? doNotReplyElem : '';
   return (
     <tr className={className}>
       <td>
-        <a href={props.contact.url}>{props.contact.full_name}</a>
+        <a href={props.contact.url}>{props.contact.full_name}</a> {doNotReply}
       </td>
       <td>{lastSms.content}</td>
       <td>{lastSms.time_received}</td>

apostello/migrations/0011_recipient_do_not_reply.py

@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.9.5 on 2016-04-28 09:06
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('apostello', '0010_auto_20160421_1411'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='recipient',
+            name='do_not_reply',
+            field=models.BooleanField(default=False, help_text='Tick this box to disable automated replies for this person.', verbose_name='Do not reply'),
+        ),
+    ]

apostello/models.py

@@ -123,6 +123,12 @@ class Recipient(models.Model):
         validators=[not_twilio_num],
         help_text="Cannot be our number, or we get an SMS loop."
     )
+    do_not_reply = models.BooleanField(
+        "Do not reply",
+        default=False,
+        help_text=
+        "Tick this box to disable automated replies for this person.",
+    )
     groups = models.ManyToManyField(RecipientGroup, blank=True)
 
     def personalise(self, message):

apostello/reply.py

@@ -120,6 +120,9 @@ def reply_to_name(self):
     def construct_reply(self):
         """Construct appropriate reply."""
 
+        if self.contact.do_not_reply:
+            return ''
+
         if self.keyword == "start":
             reply = self.reply_to_start()
         elif self.keyword == "stop":

docs/features.rst

@@ -6,6 +6,7 @@ Features
 * Track sign ups, questions, etc using keywords.
 * Manage access permissions - you can let anyone in you church have read only access.
 * Spending safety net - you can set a limit (in $) for each person. No individual SMS can be sent that will cost more than this.
+* Block auto replies to specific contacts.
 * Receive daily digest emails of incoming messages.
 * Live "wall" - curate and display incoming messages on a big screen. Great for a Q&A.
 * Post all messages to a slack channel.

tests/conftest.py

@@ -78,14 +78,21 @@ def recipients():
         last_name="Chalmers",
         number='+15005550009'
     )  # can't recieve
+    beza = Recipient.objects.create(
+        first_name="Theodore",
+        last_name="Beza",
+        number='+447927411115',
+        do_not_reply=True
+    )
 
     objs = {
         'calvin': calvin,
         'house_lamp': house_lamp,
         'knox': knox,
         'wesley': wesley,
         'john_owen': john_owen,
-        'thomas_chalmers': thomas_chalmers
+        'thomas_chalmers': thomas_chalmers,
+        'beza': beza,
     }
     return objs
 

tests/test_reply.py

@@ -93,3 +93,13 @@ def test_is_blocking_reply(self, recipients):
         )
         reply = msg.construct_reply()
         assert len(reply) == 0
+
+    def test_do_not_reply(self, recipients):
+        msg = InboundSms(
+            {
+                'From': str(recipients['beza'].number),
+                'Body': 'test'
+            }
+        )
+        reply = msg.construct_reply()
+        assert len(reply) == 0

tests/test_urls.py

@@ -332,7 +332,7 @@ def test_create_all_group_form(self, users, recipients):
         assert resp.url == '/group/all/'
         assert len(models.RecipientGroup.objects.all()) == 1
         assert models.RecipientGroup.objects.all()[0].name == 'test'
-        assert len(models.RecipientGroup.objects.all()[0].all_recipients) == 5
+        assert len(models.RecipientGroup.objects.all()[0].all_recipients) == 6
 
     def test_create_all_group_form_update(self, users, recipients, groups):
         """Test the form to create a group composed of all recipients.
@@ -346,7 +346,7 @@ def test_create_all_group_form_update(self, users, recipients, groups):
         assert resp.status_code == 302
         assert resp.url == '/group/all/'
         g = models.RecipientGroup.objects.get(name='Empty Group')
-        assert len(g.all_recipients) == 5
+        assert len(g.all_recipients) == 6
 
 
 @pytest.mark.slow