Add admin setting to toggle new application email notifications

- Add notify_new_applications boolean column to admins (default true)
- Admins can toggle this in Settings page (linked in nav)
- Super admins can also configure this for other admins
- Mailer checks preference before sending new application emails

Amp-Thread-ID: https://ampcode.com/threads/T-019c25c9-ecf2-77dc-a60d-84b7e0f3fcc8
Co-authored-by: Amp <[email protected]>

Author: leowilkin

app/controllers/admin/admins_controller.rb

@@ -79,7 +79,7 @@ def set_admin
   end
 
   def admin_params
-    permitted = [ :first_name, :last_name, :email ]
+    permitted = [ :first_name, :last_name, :email, :notify_new_applications ]
     permitted << :super_admin if current_admin.super_admin?
     params.require(:admin).permit(permitted)
   end

app/controllers/admin/settings_controller.rb

@@ -0,0 +1,21 @@
+class Admin::SettingsController < Admin::BaseController
+  def show
+    @admin = current_admin
+  end
+
+  def update
+    @admin = current_admin
+
+    if @admin.update(settings_params)
+      redirect_to admin_settings_path, notice: "Settings updated successfully."
+    else
+      render :show, status: :unprocessable_entity
+    end
+  end
+
+  private
+
+  def settings_params
+    params.require(:admin).permit(:notify_new_applications)
+  end
+end

app/mailers/admin_mailer.rb

@@ -7,10 +7,21 @@ def new_application_notification(application)
     @participant = application.participant
     @event = application.event
 
-    recipients = [ @event.contact_email ]
-    recipients += Admin.where(super_admin: true).pluck(:email)
+    recipients = []
+
+    # Add event owner if they want notifications
+    event_owner = @event.admin
+    if event_owner.notify_new_applications?
+      recipients << event_owner.email
+    end
+
+    # Add super admins who want notifications
+    recipients += Admin.where(super_admin: true, notify_new_applications: true).pluck(:email)
+
     recipients = recipients.uniq.compact
 
+    return if recipients.empty?
+
     mail(
       to: recipients,
       subject: "New Visa Letter Application - #{@event.name}"

app/views/admin/admins/_form.html.erb

@@ -40,6 +40,17 @@
     </div>
   <% end %>
 
+  <div class="border-t pt-4 mt-4">
+    <h3 class="text-sm font-medium text-gray-900 mb-3">Notification Preferences</h3>
+    <div class="flex items-center">
+      <%= f.check_box :notify_new_applications, class: "h-4 w-4 text-blue-600 focus:ring-blue-500 border-gray-300 rounded" %>
+      <%= f.label :notify_new_applications, class: "ml-2 block text-sm text-gray-700" do %>
+        <span class="font-medium">New application emails</span>
+        <span class="text-gray-500">— receive email when applications are submitted</span>
+      <% end %>
+    </div>
+  </div>
+
   <div class="flex justify-end space-x-3 pt-4 border-t">
     <%= link_to "Cancel", admin_admins_path, class: "px-4 py-2 border border-gray-300 rounded-md text-gray-700 hover:bg-gray-50" %>
     <%= f.submit admin.new_record? ? "Create Admin" : "Update Admin",

app/views/admin/settings/show.html.erb

@@ -0,0 +1,28 @@
+<div class="mb-6">
+  <h1 class="text-2xl font-bold text-gray-900">Settings</h1>
+  <p class="text-gray-500">Manage your notification preferences</p>
+</div>
+
+<div class="bg-white rounded-lg shadow p-6 max-w-2xl">
+  <%= form_with model: @admin, url: admin_settings_path, method: :patch, class: "space-y-6" do |f| %>
+    <div>
+      <h2 class="text-lg font-medium text-gray-900 mb-4">Email Notifications</h2>
+      
+      <div class="space-y-4">
+        <div class="flex items-start">
+          <div class="flex items-center h-5">
+            <%= f.check_box :notify_new_applications, class: "h-4 w-4 text-red-600 focus:ring-red-500 border-gray-300 rounded" %>
+          </div>
+          <div class="ml-3">
+            <%= f.label :notify_new_applications, "New application notifications", class: "text-sm font-medium text-gray-700" %>
+            <p class="text-sm text-gray-500">Receive an email when a new visa letter application is submitted for your events.</p>
+          </div>
+        </div>
+      </div>
+    </div>
+
+    <div class="pt-4 border-t">
+      <%= f.submit "Save Settings", class: "bg-red-600 text-white py-2 px-4 rounded-md hover:bg-red-700 cursor-pointer" %>
+    </div>
+  <% end %>
+</div>

app/views/layouts/admin.html.erb

@@ -43,6 +43,7 @@
                 <span class="bg-yellow-500 text-yellow-900 text-xs px-2 py-1 rounded ml-1 font-semibold">Super Admin</span>
               <% end %>
             </span>
+            <%= link_to "Settings", admin_settings_path, class: "text-gray-300 hover:text-white text-sm hover:underline" %>
             <%= link_to "View Site", root_path, class: "text-gray-300 hover:text-white text-sm hover:underline" %>
             <%= button_to "Sign Out", destroy_admin_session_path, method: :delete, class: "text-gray-300 hover:text-white text-sm hover:underline" %>
           </div>

config/routes.rb

@@ -50,6 +50,8 @@
   namespace :admin do
     get "/", to: "dashboard#index", as: :dashboard
 
+    resource :settings, only: [ :show, :update ]
+
     resources :events
     resources :visa_letter_applications, only: [ :index, :show ] do
       member do

db/migrate/20260203000002_add_email_notifications_to_admins.rb

@@ -0,0 +1,5 @@
+class AddEmailNotificationsToAdmins < ActiveRecord::Migration[8.1]
+  def change
+    add_column :admins, :notify_new_applications, :boolean, default: true, null: false
+  end
+end