Added source field to Thank You Note

openstax · Nov 17, 2023 · 38be7ae · 38be7ae
1 parent 8e62c07
commit 38be7ae
Show file tree

Hide file tree

Showing 7 changed files with 58 additions and 4 deletions.
diff --git a/donations/migrations/0008_thankyounote_source.py b/donations/migrations/0008_thankyounote_source.py
@@ -0,0 +1,18 @@
+# Generated by Django 4.1.7 on 2023-11-17 14:37
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('donations', '0007_auto_20220516_2113'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='thankyounote',
+            name='source',
+            field=models.CharField(blank=True, default='', max_length=255),
+        ),
+    ]
diff --git a/donations/models.py b/donations/models.py
@@ -21,6 +21,7 @@ class ThankYouNote(models.Model):
     created = models.DateField(auto_now_add=True)
     consent_to_share_or_contact = models.BooleanField(default=False)
     contact_email_address = models.EmailField(blank=True, null=True)
+    source = models.CharField(max_length=255, default="", blank=True)
 
 class DonationPopup(models.Model):
     download_image = models.ImageField(null=True, blank=True)

diff --git a/donations/serializers.py b/donations/serializers.py
@@ -11,14 +11,16 @@ class Meta:
                   'institution',
                   'created',
                   'consent_to_share_or_contact',
-                  'contact_email_address')
+                  'contact_email_address',
+                  'source')
         read_only_fields = ('thank_you_note',
                             'first_name',
                             'last_name',
                             'institution',
                             'created',
                             'consent_to_share_or_contact',
-                            'contact_email_address')
+                            'contact_email_address',
+                            'source')
 
 
 class DonationPopupSerializer(serializers.ModelSerializer):

diff --git a/donations/tests.py b/donations/tests.py
@@ -37,7 +37,7 @@ def test_donation_api_get(self):
 class ThankYouNoteTest(APITestCase, TestCase):
 
     def test_thank_you_note_api_post(self):
-        data = {"thank_you_note":"OpenStax is the best! Loved not paying for a book", "last_name": "Drew", "first_name": "Jessica", "institution": "Rice University", "consent_to_share_or_contact": "True", "contact_email_address": "[email protected]"}
+        data = {"thank_you_note":"OpenStax is the best! Loved not paying for a book", "last_name": "Drew", "first_name": "Jessica", "institution": "Rice University", "consent_to_share_or_contact": "True", "contact_email_address": "[email protected]", "source": "PDF download"}
         response = self.client.post('/apps/cms/api/donations/thankyounote/', data, format='json')
         self.assertEqual(response.status_code, status.HTTP_201_CREATED)
         tyn = ThankYouNote.objects.filter(last_name='Drew').values()

diff --git a/donations/views.py b/donations/views.py
@@ -17,13 +17,15 @@ def post(self, request):
         institution = request.data['institution']
         consent_to_share_or_contact = request.data.get('consent_to_share_or_contact', False)
         contact_email_address = request.data.get('contact_email_address', '')
+        source = request.data.get('source', '')
 
         ty_note = ThankYouNote.objects.create(thank_you_note=thank_you_note,
                                               first_name=first_name,
                                               last_name=last_name,
                                               institution=institution,
                                               consent_to_share_or_contact=consent_to_share_or_contact,
-                                              contact_email_address=contact_email_address)
+                                              contact_email_address=contact_email_address,
+                                              source=source)
 
         serializer = ThankYouNoteSerializer(data=request.data)
         if serializer.is_valid():

diff --git a/errata/migrations/0056_alter_errata_options.py b/errata/migrations/0056_alter_errata_options.py
@@ -0,0 +1,17 @@
+# Generated by Django 4.1.7 on 2023-11-17 14:37
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('errata', '0055_remove_errata_accounts_user_email_and_more'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='errata',
+            options={'verbose_name': 'erratum list', 'verbose_name_plural': 'errata list'},
+        ),
+    ]
diff --git a/snippets/migrations/0032_merge_20231117_0830.py b/snippets/migrations/0032_merge_20231117_0830.py
@@ -0,0 +1,14 @@
+# Generated by Django 4.1.7 on 2023-11-17 14:30
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('snippets', '0031_merge_20231101_1313'),
+        ('snippets', '0031_merge_20231101_1438'),
+    ]
+
+    operations = [
+    ]