Merge pull request #344 from NHSDigital/PPHA-589-add-yearly-option-to-smoking-frequency

PPHA-589: Add yearly option to smoking frequency

Author: Themitchell

lung_cancer_screening/questions/forms/smoking_frequency_form.py

@@ -31,7 +31,11 @@ def __init__(self, *args, tobacco_smoking_history, normal_tobacco_smoking_histor
         )
         value_field.add_hint_for_choice(
             SmokingFrequencyValues.MONTHLY,
-            "Select this option if you smoke once a month, or once every few months",
+            "Select this option if you smoke at least once a month",
+        )
+        value_field.add_hint_for_choice(
+            SmokingFrequencyValues.YEARLY,
+            "For example, 2 to 3 times a year or fewer",
         )
 
     class Meta:

lung_cancer_screening/questions/migrations/0006_alter_smokingfrequencyresponse_value.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.2.11 on 2026-03-09 14:25
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('questions', '0005_alter_tobaccosmokinghistory_type'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='smokingfrequencyresponse',
+            name='value',
+            field=models.CharField(choices=[('D', 'Daily'), ('W', 'Weekly'), ('M', 'Monthly'), ('Y', 'Yearly')], max_length=1),
+        ),
+    ]

lung_cancer_screening/questions/models/smoking_frequency_response.py

@@ -8,12 +8,14 @@ class SmokingFrequencyValues(models.TextChoices):
     DAILY = "D", "Daily"
     WEEKLY = "W", "Weekly"
     MONTHLY = "M", "Monthly"
+    YEARLY = "Y", "Yearly"
 
 class SmokingFrequencyResponse(BaseModel):
     SINGULAR_TEXT_MAP = {
         SmokingFrequencyValues.DAILY: "day",
         SmokingFrequencyValues.WEEKLY: "week",
         SmokingFrequencyValues.MONTHLY: "month",
+        SmokingFrequencyValues.YEARLY: "year",
     }
 
     tobacco_smoking_history = models.OneToOneField(