Merge pull request #34 from turingschool/jm/disable_inactive

Gemfile

@@ -57,3 +57,7 @@ group :development do
   # Speed up commands on slow machines / big apps [https://github.com/rails/spring]
   # gem "spring"
 end
+
+gem 'redis'
+gem 'sidekiq'
+gem 'sidekiq-cron'

Gemfile.lock

@@ -104,6 +104,8 @@ GEM
     docile (1.4.1)
     drb (2.2.1)
     erubi (1.13.0)
+    et-orbi (1.2.11)
+      tzinfo
     factory_bot (6.4.6)
       activesupport (>= 5.0.0)
     factory_bot_rails (6.4.3)
@@ -116,6 +118,9 @@ GEM
       logger
     faraday-net_http (3.1.1)
       net-http
+    fugit (1.11.1)
+      et-orbi (~> 1, >= 1.2.11)
+      raabro (~> 1.4)
     globalid (1.2.1)
       activesupport (>= 6.1)
     i18n (1.14.5)
@@ -175,6 +180,7 @@ GEM
     public_suffix (6.0.1)
     puma (6.4.2)
       nio4r (~> 2.0)
+    raabro (1.4.0)
     racc (1.8.1)
     rack (3.1.7)
     rack-cors (2.0.2)
@@ -218,6 +224,10 @@ GEM
     rake (13.2.1)
     rdoc (6.7.0)
       psych (>= 4.0.0)
+    redis (5.3.0)
+      redis-client (>= 0.22.0)
+    redis-client (0.22.2)
+      connection_pool
     regexp_parser (2.9.2)
     reline (0.5.9)
       io-console (~> 0.5)
@@ -240,6 +250,16 @@ GEM
     rspec-support (3.13.1)
     shoulda-matchers (6.4.0)
       activesupport (>= 5.2.0)
+    sidekiq (7.3.1)
+      concurrent-ruby (< 2)
+      connection_pool (>= 2.3.0)
+      logger
+      rack (>= 2.2.4)
+      redis-client (>= 0.22.2)
+    sidekiq-cron (1.12.0)
+      fugit (~> 1.8)
+      globalid (>= 1.0.1)
+      sidekiq (>= 6)
     simplecov (0.22.0)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
@@ -282,8 +302,11 @@ DEPENDENCIES
   puma (>= 5.0)
   rack-cors
   rails (~> 7.1.3, >= 7.1.3.4)
+  redis
   rspec-rails (~> 6.1)
   shoulda-matchers (~> 6.4)
+  sidekiq
+  sidekiq-cron
   simplecov (~> 0.22.0)
   tzinfo-data
 

Procfile

@@ -0,0 +1,2 @@
+web: bundle exec puma -C config/puma.rb
+worker: bundle exec sidekiq -C config/sidekiq.yml

app/models/link.rb

@@ -1,6 +1,8 @@
 require 'securerandom'
 
 class Link < ApplicationRecord
+  default_scope { where(disabled: false) }
+
   validates_presence_of :user_id
   validates_presence_of :original
   validates :short, uniqueness: true, presence: true

app/sidekiq/disable_inactive_links_job.rb

@@ -0,0 +1,14 @@
+class DisableInactiveLinksJob
+  include Sidekiq::Worker
+  queue_as :default
+
+  def perform
+    links = Link.all
+    six_months_ago = Time.current - 15552000 #this is 180 days in seconds
+    links.each do |link|
+      if link.click_count < 50 && link.last_click != nil && link.last_click < six_months_ago
+        link.update(disabled: true)
+      end
+    end
+  end
+end

config/initializers/sidekiq.rb

@@ -0,0 +1,8 @@
+require 'sidekiq'
+require 'sidekiq/cron/job'
+
+Sidekiq::Cron::Job.create(
+  name: 'Disable inactive links daily',
+  cron: '0 0 * * *', # This means daily at midnight
+  class: 'DisableInactiveLinksJob'
+)

config/routes.rb

@@ -4,6 +4,7 @@
   # Reveal health status on /up that returns 200 if the app boots with no exceptions, otherwise 500.
   # Can be used by load balancers and uptime monitors to verify that the app is live.
   get 'up' => 'rails/health#show', as: :rails_health_check
+  require 'sidekiq/web'
 
   namespace :api do
     namespace :v1 do
@@ -16,4 +17,6 @@
       get 'top_links', to: 'links#top_links'
     end
   end
+
+  mount Sidekiq::Web => '/sidekiq'
 end

config/sidekiq.yml

@@ -0,0 +1,3 @@
+:concurrency: 5
+:queues:
+  - default

db/migrate/20240830004510_add_disabled_to_links.rb

@@ -0,0 +1,5 @@
+class AddDisabledToLinks < ActiveRecord::Migration[7.1]
+  def change
+    add_column :links, :disabled, :boolean, default: false
+  end
+end