Prevents inactive repos from beeing suggested in the poke_inactive mail

Author: hbontempo-br

app/mailers/user_mailer.rb

@@ -77,12 +77,12 @@ def send_triage(user:, assignment:, repo:, create: false)
 
   def poke_inactive(user:, repos_by_need_ids:)
     return unless set_and_check_user(user)
-    @most_repo = Repo.order_by_issue_count.first
+    @most_repo = Repo.active.order_by_issue_count.first
 
     repo_need_id = repos_by_need_ids.detect { |id| id != @most_repo.id }
     @need_repo = Repo.where(id: repo_need_id).first
 
-    @random_repo = Repo.rand.not_in(@most_repo.id, @need_repo.id).first || @most_repo || @need_repo
+    @random_repo = Repo.active.rand.not_in(@most_repo.id, @need_repo.id).first || @most_repo || @need_repo
     mail(to: @user.email, reply_to: "[email protected]", subject: "CodeTriage misses you")
   end
 

lib/tasks/schedule.rake

@@ -61,7 +61,7 @@ namespace :schedule do
   task poke_inactive: :environment do
     next unless Date.today.tuesday?
 
-    repos_by_need_ids = Repo.order_by_need.limit(10).pluck(:id)
+    repos_by_need_ids = Repo.active.order_by_need.limit(10).pluck(:id)
     User.inactive.find_each(batch_size: 100) do |user|
       BackgroundInactiveEmailJob.perform_later(user, repos_by_need_ids: repos_by_need_ids)
     end