Merge branch 'main' into 4947-update-default-email-text

Author: jasonwang7517

Gemfile

@@ -184,6 +184,12 @@ group :development do
   gem "rails-erd"
   # Allows to create a console in the browser.
   gem "web-console"
+  # Middleware that displays speed badge for every HTML page
+  gem "rack-mini-profiler"
+  # Adds memory profiling to rack-mini-profiler
+  gem "memory_profiler"
+  # Adds call-stack profiling flamegraphs to rack-mini-profiler
+  gem "stackprof"
 end
 
 group :test do

Gemfile.lock

@@ -370,6 +370,7 @@ GEM
       net-smtp
     marcel (1.0.4)
     matrix (0.4.2)
+    memory_profiler (1.1.0)
     method_source (1.1.0)
     mini_magick (4.13.2)
     mini_mime (1.1.5)
@@ -494,6 +495,8 @@ GEM
       nio4r (~> 2.0)
     racc (1.8.1)
     rack (3.1.8)
+    rack-mini-profiler (3.3.1)
+      rack (>= 1.2.0)
     rack-protection (4.0.0)
       base64 (>= 0.1.0)
       rack (>= 3.0.0, < 4)
@@ -657,6 +660,7 @@ GEM
       actionpack (>= 5.2)
       activesupport (>= 5.2)
       sprockets (>= 3.0.0)
+    stackprof (0.2.27)
     standard (1.43.0)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.0)
@@ -775,6 +779,7 @@ DEPENDENCIES
   lograge
   magic_test
   matrix
+  memory_profiler
   money-rails
   newrelic_rpm
   nokogiri (>= 1.10.4)
@@ -791,6 +796,7 @@ DEPENDENCIES
   pry-rails
   pry-remote
   puma
+  rack-mini-profiler
   rails (= 7.2.2)
   rails-controller-testing
   rails-erd
@@ -806,6 +812,7 @@ DEPENDENCIES
   simplecov
   solid_cache (~> 1.0)
   sprockets (~> 4.2.1)
+  stackprof
   standard (~> 1.40)
   standard-performance
   standard-rails

app/javascript/application.js

@@ -35,6 +35,11 @@ import 'utils/purchases'
 
 import Rails from "@rails/ujs"
 Rails.start()
+
+// Initialize Active Storage
+import * as ActiveStorage from "@rails/activestorage";
+ActiveStorage.start();
+
 // Disable turbo by default to avoid issues with turbolinks
 Turbo.session.drive = false
 
@@ -107,4 +112,3 @@ $(document).ready(function(){
     });
     picker.setDateRange(startDate, endDate);
 });
-

app/javascript/controllers/file_input_label_controller.js

@@ -0,0 +1,49 @@
+import { Controller } from "@hotwired/stimulus"
+
+// Connects to data-controller="file-input-label"
+//
+// Reproduces the native browser behavior of updating a file input label
+// to show the selected file's name. This is necessary when using a custom
+// file input, such as with Bootstrap, that does not update automatically.
+//
+// Key Features:
+// 1. Handles initial display of a default label text (e.g., "Choose file..." or
+//    the previously selected file name if present).
+// 2. Updates the label dynamically when a new file is selected.
+//
+// How it works:
+// - When a file is selected, the `fileSelected` method updates the text of the
+//   label to reflect the name of the selected file.
+// - On page load, the `connect` method ensures the label is initialized to the
+//   correct state (default text or file name, if a file was previously selected).
+//
+// This controller is used in coordination with direct uploads in Active Storage.
+// When a validation error occurs, previously selected files persist on the server
+// (via direct upload), and the file name can be displayed to the user.
+export default class extends Controller {
+  static targets = ["input", "label"];
+  static values = {
+    defaultText: { type: String, default: 'Choose file...' }
+  }
+
+  connect() {
+    this.updateLabel();
+  }
+
+  updateLabel() {
+    const input = this.inputTarget;
+    const label = this.labelTarget;
+
+    // Check if the file input has a file selected
+    if (input.files.length > 0) {
+      label.textContent = input.files[0].name;
+    } else {
+      label.textContent = this.defaultTextValue;
+    }
+  }
+
+  // Update the label when a file is selected
+  fileSelected() {
+    this.updateLabel();
+  }
+}

app/views/partners/_documents.html.erb

@@ -1,6 +1,6 @@
 <div class="card card-info card-outline">
   <div class="card-header">
-    <h2 class="card-title">Documents</h2>
+    <h2 class="card-title fs-3">Documents</h2>
   </div>
   <div class="card-body p-0">
     <div class="tab-content" id="custom-tabs-three-tabContent">

app/views/partners/_show_header.html.erb

@@ -1,6 +1,6 @@
 <div class="card card-info card-outline">
   <div class="card-header">
-    <h2 class="card-title"><%= partner.name %></h2>
+    <h2 class="card-title fs-3"><%= partner.name %></h2>
   </div>
   <div class="card-body p-0">
     <div class="row">

app/views/partners/_uninvited_header.html.erb

@@ -1,6 +1,6 @@
 <div class="card card-info card-outline">
   <div class="card-header">
-    <h2 class="card-title"><%= partner.name %></h2>
+    <h2 class="card-title fs-3"><%= partner.name %></h2>
   </div>
   <div class="card-body p-0">
   </div>

app/views/partners/profiles/edit/_agency_information.html.erb

@@ -10,20 +10,14 @@
             <%= form.input :name, label: "Agency Name", class: "form-control", wrapper: :input_group %>
             <%= profile_form.input :agency_type, collection: Partner::AGENCY_TYPES.values, label: "Agency Type", class: "form-control", wrapper: :input_group %>
             <%= profile_form.input :other_agency_type, label: "Other Agency Type", class: "form-control", wrapper: :input_group %>
-            <div class="form-group row">
-              <label class="control-label col-md-3">501(c)(3) IRS Determination Letter or other Proof of Agency Status</label>
-              <% if profile.proof_of_partner_status.attached? %>
-                <div class="col-md-8">
-                  Attached
-                  file: <%= link_to profile.proof_of_partner_status.blob['filename'], rails_blob_path(profile.proof_of_partner_status), class: "font-weight-bold" %>
-                  <%= profile_form.file_field :proof_of_partner_status, class: "form-control-file form-control" %>
-                </div>
-              <% else %>
-                <div class="col-md-8">
-                  <%= profile_form.file_field :proof_of_partner_status, class: "form-control-file" %>
-                </div>
-              <% end %>
-            </div>
+
+            <%= render "shared/custom_file_input",
+              form_builder: profile_form,
+              attachment: profile.proof_of_partner_status,
+              attachment_name: :proof_of_partner_status,
+              label_for: "partner_profile_proof_of_partner_status",
+              label_text: "501(c)(3) IRS Determination Letter or other Proof of Agency Status" %>
+
             <%= profile_form.input :agency_mission, label: "Agency Mission", class: "form-control", wrapper: :input_group %>
             <%= profile_form.input :address1, label: "Address (line 1)", class: "form-control", wrapper: :input_group %>
             <%= profile_form.input :address2, label: "Address (line 2)", class: "form-control", wrapper: :input_group %>

app/views/partners/profiles/edit/_agency_stability.html.erb

@@ -10,18 +10,14 @@
             <%= form.input :founded, label: "Year Founded", class: "form-control", wrapper: :input_group %>
             <%= form.input :form_990, label: "Form 990 Filed", as: :radio_buttons, class: "form-control",
                            wrapper: :input_group, wrapper_html: {class: "form-yesno"}, input_html: {class: "radio-yesno"} %>
-            <label class="control-label col-md-3">Form 990</label>
-            <% if profile.proof_of_form_990.attached? %>
-              <div class="col-md-8">
-                Attached
-                file: <%= link_to profile.proof_of_form_990.blob['filename'], rails_blob_path(profile.proof_of_form_990), class: "font-weight-bold" %>
-                <%= form.file_field :proof_of_form_990, class: "form-control-file form-control" %>
-              </div>
-            <% else %>
-              <div class="col-md-8">
-                <%= form.file_field :proof_of_form_990, class: "form-control-file" %>
-              </div>
-            <% end %>
+
+            <%= render "shared/custom_file_input",
+              form_builder: form,
+              attachment: profile.proof_of_form_990,
+              attachment_name: :proof_of_form_990,
+              label_for: "partner_profile_proof_of_form_990",
+              label_text: "Form 990" %>
+
             <%= form.input :program_name, label: "Program Name(s)", class: "form-control", wrapper: :input_group %>
             <%= form.input :program_description, label: "Program Description(s)", class: "form-control", wrapper: :input_group %>
             <%= form.input :program_age, label: "Agency Age", class: "form-control", wrapper: :input_group %>

app/views/partners/profiles/step/_agency_information_form.html.erb

@@ -11,19 +11,12 @@
     <%= pf.input :other_agency_type, label: "Other Agency Type", class: "form-control" %>
   </div>
 
-  <div class="form-group row">
-    <label class="control-label col-md-3">501(c)(3) IRS Determination Letter or other Proof of Agency Status</label>
-    <% if profile.proof_of_partner_status.attached? %>
-      <div class="col-md-8">
-        Attached file: <%= link_to profile.proof_of_partner_status.blob['filename'], rails_blob_path(profile.proof_of_partner_status), class: "font-weight-bold" %>
-        <%= pf.file_field :proof_of_partner_status, class: "form-control-file" %>
-      </div>
-    <% else %>
-      <div class="col-md-8">
-        <%= pf.file_field :proof_of_partner_status, class: "form-control-file" %>
-      </div>
-    <% end %>
-  </div>
+  <%= render "shared/custom_file_input",
+           form_builder: pf,
+           attachment: profile.proof_of_partner_status,
+           attachment_name: :proof_of_partner_status,
+           label_for: "partner_profile_proof_of_partner_status",
+           label_text: "501(c)(3) IRS Determination Letter or other Proof of Agency Status" %>
 
   <div class="form-group">
     <%= pf.input :agency_mission, as: :text, label: "Agency Mission", class: "form-control" %>