Exclude students with no attendance in the past year from chaser email

Narrows the chaser target to members who attended between 3 months and
1 year ago, filtering out long-inactive students who are unlikely to
re-engage.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 2 people

app/services/three_month_email_service.rb

@@ -2,20 +2,29 @@
 
 class ThreeMonthEmailService
   def self.send_chaser
-    cutoff = 3.months.ago.beginning_of_day
+    three_month_cutoff = 3.months.ago.beginning_of_day
+    one_year_cutoff = 1.year.ago.beginning_of_day
+
     recent_attendee_ids = WorkshopInvitation.to_students
                                             .attended
                                             .joins(:workshop)
-                                            .where('workshops.date_and_time >= ?', cutoff)
+                                            .where('workshops.date_and_time >= ?', three_month_cutoff)
                                             .select(:member_id)
 
+    past_year_attendee_ids = WorkshopInvitation.to_students
+                                               .attended
+                                               .joins(:workshop)
+                                               .where('workshops.date_and_time >= ?', one_year_cutoff)
+                                               .select(:member_id)
+
     members = Member.not_banned
                     .accepted_toc
                     .joins(:groups)
                     .merge(Group.students)
                     .left_joins(:member_email_deliveries)
                     .where(member_email_deliveries: { id: nil })
                     .where.not(id: recent_attendee_ids)
+                    .where(id: past_year_attendee_ids)
                     .distinct
     return if members.empty?
 

spec/services/three_month_email_service_spec.rb

@@ -9,6 +9,13 @@
     let!(:eligible_student) do
       member = Fabricate(:member)
       Fabricate(:subscription, member: member, group: students_group)
+      Fabricate(
+        :workshop_invitation,
+        member: member,
+        workshop: Fabricate(:workshop, chapter: chapter, date_and_time: 6.months.ago),
+        role: "Student",
+        attended: true
+      )
       member
     end
 
@@ -63,6 +70,19 @@
       member
     end
 
+    let!(:student_with_very_old_attendance) do
+      member = Fabricate(:member)
+      Fabricate(:subscription, member: member, group: students_group)
+      Fabricate(
+        :workshop_invitation,
+        member: member,
+        workshop: Fabricate(:workshop, chapter: chapter, date_and_time: 14.months.ago),
+        role: "Student",
+        attended: true
+      )
+      member
+    end
+
     it "emails only students who have not attended in the last 3 months and were not emailed before" do
       expect { perform_enqueued_jobs { call } }.to change(MemberEmailDelivery, :count).by(2)
 
@@ -94,6 +114,12 @@
       expect(MemberEmailDelivery.where(member: student_without_toc)).to be_empty
     end
 
+    it "does not email students who have not attended in the past year" do
+      perform_enqueued_jobs { call }
+
+      expect(MemberEmailDelivery.where(member: student_with_very_old_attendance)).to be_empty
+    end
+
     it "sends only one chaser for a member with multiple student subscriptions" do
       member = Fabricate(:member)
       other_chapter = Fabricate(:chapter)
@@ -151,6 +177,13 @@
     it "emails a student member who has recent attendance only as a coach" do
       member = Fabricate(:member)
       Fabricate(:subscription, member: member, group: students_group)
+      Fabricate(
+        :workshop_invitation,
+        member: member,
+        workshop: Fabricate(:workshop, chapter: chapter, date_and_time: 6.months.ago),
+        role: "Student",
+        attended: true
+      )
       Fabricate(
         :workshop_invitation,
         member: member,