feat (project): add validations for banner and preprocess variants

Author: cskartikey

Gemfile

@@ -38,7 +38,7 @@ gem "kamal", require: false
 gem "thruster", require: false
 
 # Use Active Storage variants [https://guides.rubyonrails.org/active_storage_overview.html#transforming-images]
-# gem "image_processing", "~> 1.2"
+gem "image_processing", "~> 1.2"
 
 group :development, :test do
   # See https://guides.rubyonrails.org/debugging_rails_applications.html#debugging-with-the-debug-gem
@@ -86,3 +86,5 @@ gem "slack-ruby-client"
 gem "tailwindcss-ruby", "~> 4.1"
 
 gem "tailwindcss-rails", "~> 4.3"
+
+gem "active_storage_validations"

Gemfile.lock

@@ -46,6 +46,12 @@ GEM
       erubi (~> 1.11)
       rails-dom-testing (~> 2.2)
       rails-html-sanitizer (~> 1.6)
+    active_storage_validations (3.0.2)
+      activejob (>= 6.1.4)
+      activemodel (>= 6.1.4)
+      activestorage (>= 6.1.4)
+      activesupport (>= 6.1.4)
+      marcel (>= 1.0.3)
     activejob (8.0.3)
       activesupport (= 8.0.3)
       globalid (>= 0.3.6)
@@ -153,6 +159,13 @@ GEM
       multipart-post (~> 2.0)
     faraday-net_http (3.4.1)
       net-http (>= 0.5.0)
+    ffi (1.17.2-aarch64-linux-gnu)
+    ffi (1.17.2-aarch64-linux-musl)
+    ffi (1.17.2-arm-linux-gnu)
+    ffi (1.17.2-arm-linux-musl)
+    ffi (1.17.2-arm64-darwin)
+    ffi (1.17.2-x86_64-linux-gnu)
+    ffi (1.17.2-x86_64-linux-musl)
     fiddle (1.1.8)
     fugit (1.11.2)
       et-orbi (~> 1, >= 1.2.11)
@@ -164,6 +177,9 @@ GEM
     hashie (5.0.0)
     i18n (1.14.7)
       concurrent-ruby (~> 1.0)
+    image_processing (1.14.0)
+      mini_magick (>= 4.9.5, < 6)
+      ruby-vips (>= 2.0.17, < 3)
     importmap-rails (2.2.2)
       actionpack (>= 6.0.0)
       activesupport (>= 6.0.0)
@@ -209,6 +225,8 @@ GEM
       net-smtp
     marcel (1.1.0)
     matrix (0.4.3)
+    mini_magick (5.3.1)
+      logger
     mini_mime (1.1.5)
     minitest (5.26.0)
     msgpack (1.8.0)
@@ -392,6 +410,9 @@ GEM
       rubocop-performance (>= 1.24)
       rubocop-rails (>= 2.30)
     ruby-progressbar (1.13.0)
+    ruby-vips (2.2.5)
+      ffi (~> 1.12)
+      logger
     rubyzip (3.1.1)
     securerandom (0.4.1)
     selenium-webdriver (4.35.0)
@@ -500,6 +521,7 @@ PLATFORMS
 
 DEPENDENCIES
   aasm
+  active_storage_validations
   annotaterb
   blind_index
   bootsnap
@@ -508,6 +530,7 @@ DEPENDENCIES
   capybara
   debug
   erb_lint
+  image_processing (~> 1.2)
   importmap-rails
   jbuilder
   kamal

app/models/project.rb

@@ -17,8 +17,34 @@ class Project < ApplicationRecord
     has_many :users, through: :memberships
 
     has_one_attached :demo_video
-    has_one_attached :banner
+    # https://github.com/rails/rails/pull/39135
+    has_one_attached :banner do |attachable|
+        # using resize_to_fill instead of resize_to_limit because consistency. might change it to resize_to_limit
+        # we're preprocessing them because its likely going to be used
+
+        # for explore and projects#index
+        attachable.variant :card,
+                           resize_to_fill: [ 1600, 900 ],
+                           format: :webp,
+                           preprocess: true,
+                           saver: { strip: true, quality: 75 }
+
+        #   attachable.variant :not_sure,
+        #     resize_to_fill: [ 1200, 630 ],
+        #     format: :webp,
+        #     saver: { strip: true, quality: 75 }
+
+        # for voting
+        attachable.variant :thumb,
+                           resize_to_fill: [ 400, 210 ],
+                           format: :webp,
+                           preprocess: true,
+                           saver: { strip: true, quality: 75 }
+    end
 
     validates :title, presence: true, length: { maximum: 120 }
     validates :description, length: { maximum: 1_000 }, allow_blank: true
+    validates :banner,
+              content_type: [ "image/jpeg", "image/png", "image/webp", "image/heic", "image/heif" ],
+              size: { less_than: 10.megabytes, message: "is too large (max 10 MB)" }
 end