rails_error_dashboard 0.6.2 → 0.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d065c1edba3f66205914ba82dbfc5aed936e0df62fd96df8f1dcd1667bc119af
-  data.tar.gz: dff67e2453ec7d33429b5a4c07754b1e7d9c3103946cd5fdcb6ed282a3a14bce
+  metadata.gz: 938ab24cad38b8b20bc0dd0f2a1eb6e5c9e46a4634fb97e2c01b3487221c9fc3
+  data.tar.gz: 6e053a3f8c41e3c3f75f95ab1f9bd8908e9bcea4c473e2d1d3a0c11b7bdf0bdb
 SHA512:
-  metadata.gz: 14b55055cb0a3c765824778c82cb7d8f33cd2d4ad5c445ea2e568fc10c51aec611271e0610fa295c8318a4701bba6a051db5f313ee6f12b4d6ac04be331df85f
-  data.tar.gz: 0c154f205b6622ad747e3e037864b0e2bc2078f72098f71c2b26fb42d717a1b09a275e87ec29f2e34864684505b66b169f96efe8cc13910dc6a9f4c58fca5a4c
+  metadata.gz: 2b84b4a4f917863e7577e253f2324100a06b6fef6f33d56cef211f28a4f1dc4688932f618489f19924baa8aa065135cfcaeffb2c7431f15bea898d3e334395b7
+  data.tar.gz: 4149d8da317c77751e04db12b46f02924fe46e1a420075232e3fa7c96f12218166249596b33a55f1790e696b5fca3580cd5da62d38a39585e356660a702a518c

data/app/controllers/rails_error_dashboard/errors_controller.rb

@@ -240,7 +240,7 @@ module RailsErrorDashboard
           resolution_comment: params[:resolution_comment]
         )
       when "mute"
-        Commands::BatchMuteErrors.call(error_ids, muted_by: params[:muted_by])
+        Commands::BatchMuteErrors.call(error_ids, muted_by: params[:muted_by], reason: params[:reason])
       when "unmute"
         Commands::BatchUnmuteErrors.call(error_ids)
       when "delete"

data/app/views/rails_error_dashboard/errors/_user_errors_table.html.erb

@@ -61,7 +61,7 @@
             </td>
           <% end %>
           <td>
-            <%= link_to "View Errors", errors_path(user_id: user_data[:user_id]), class: "btn btn-sm btn-outline-primary" %>
+            <%= link_to "View Errors", errors_path(user_id: user_data[:user_id], unresolved: '0'), class: "btn btn-sm btn-outline-primary" %>
           </td>
         </tr>
       <% end %>

data/app/views/rails_error_dashboard/errors/analytics.html.erb

@@ -148,7 +148,7 @@
             <div class="d-flex gap-2 flex-wrap">
               <small class="text-muted me-2">Quick Links:</small>
               <% @errors_by_platform.keys.each do |platform| %>
-                <%= link_to platform, errors_path(platform: platform), class: "btn btn-sm btn-outline-secondary" %>
+                <%= link_to platform, errors_path(platform: platform, unresolved: '0'), class: "btn btn-sm btn-outline-secondary" %>
               <% end %>
             </div>
           </div>
@@ -304,7 +304,7 @@
                       </div>
                     </td>
                     <td>
-                      <%= link_to "View Errors", errors_path(error_type: error_type), class: "btn btn-sm btn-outline-primary" %>
+                      <%= link_to "View Errors", errors_path(error_type: error_type, unresolved: '0'), class: "btn btn-sm btn-outline-primary" %>
                     </td>
                   </tr>
                 <% end %>
@@ -587,7 +587,7 @@
                       </td>
                       <td><strong><%= hours %> hours</strong></td>
                       <td>
-                        <%= link_to "View", errors_path(severity: severity), class: "btn btn-sm btn-outline-primary" %>
+                        <%= link_to "View", errors_path(severity: severity, unresolved: '0'), class: "btn btn-sm btn-outline-primary" %>
                       </td>
                     </tr>
                   <% end %>

data/app/views/rails_error_dashboard/errors/correlation.html.erb

@@ -105,7 +105,7 @@
                     <% end %>
                   </td>
                   <td>
-                    <%= link_to "View", errors_path(search: release[:version]), class: "btn btn-sm btn-outline-primary" %>
+                    <%= link_to "View", errors_path(search: release[:version], unresolved: '0'), class: "btn btn-sm btn-outline-primary" %>
                   </td>
                 </tr>
               <% end %>

data/app/views/rails_error_dashboard/errors/index.html.erb

@@ -89,8 +89,8 @@
         <%= link_to errors_path(app_context.merge(unresolved: '0')), class: "btn filter-pill #{current_status.blank? && !is_unresolved_only ? 'active btn-primary' : 'btn-outline-primary'}", style: "text-decoration: none;" do %>All<% end %>
         <%= link_to errors_path(app_context.merge(unresolved: '1')), class: "btn filter-pill #{is_unresolved_only && current_status.blank? ? 'active btn-primary' : 'btn-outline-primary'}", style: "text-decoration: none;" do %>Unresolved <span style="font-size: 11px; opacity: 0.7;"><%= @stats[:unresolved] %></span><% end %>
         <%= link_to errors_path(app_context.merge(status: 'resolved', unresolved: '0')), class: "btn filter-pill #{current_status == 'resolved' ? 'active btn-primary' : 'btn-outline-primary'}", style: "text-decoration: none;" do %>Resolved<% end %>
-        <%= link_to errors_path(app_context.merge(assigned_to: '__assigned__')), class: "btn filter-pill #{params[:assigned_to] == '__assigned__' ? 'active btn-primary' : 'btn-outline-primary'}", style: "text-decoration: none;" do %>Assigned<% end %>
-        <%= link_to errors_path(app_context.merge(reopened: 'true')), class: "btn filter-pill #{params[:reopened] == 'true' ? 'active btn-primary' : 'btn-outline-primary'}", style: "text-decoration: none;" do %>Reopened<% end %>
+        <%= link_to errors_path(app_context.merge(assigned_to: '__assigned__', unresolved: '0')), class: "btn filter-pill #{params[:assigned_to] == '__assigned__' ? 'active btn-primary' : 'btn-outline-primary'}", style: "text-decoration: none;" do %>Assigned<% end %>
+        <%= link_to errors_path(app_context.merge(reopened: 'true', unresolved: '0')), class: "btn filter-pill #{params[:reopened] == 'true' ? 'active btn-primary' : 'btn-outline-primary'}", style: "text-decoration: none;" do %>Reopened<% end %>
       </div>
 
       <div style="width: 1px; height: 24px; background: var(--border-primary);"></div>

data/app/views/rails_error_dashboard/errors/platform_comparison.html.erb

@@ -88,7 +88,7 @@
               </div>
             </div>
             <div class="card-footer border-top">
-              <%= link_to "View #{platform} Errors", errors_path(platform: platform), class: "btn btn-sm btn-outline-primary w-100" %>
+              <%= link_to "View #{platform} Errors", errors_path(platform: platform, unresolved: '0'), class: "btn btn-sm btn-outline-primary w-100" %>
             </div>
           </div>
         </div>

data/app/views/rails_error_dashboard/errors/show.html.erb

@@ -203,15 +203,15 @@
           <span style="font-size: 13px; font-weight: 600;">Quick Actions</span>
         </div>
         <div style="padding: var(--space-3) var(--space-5); display: flex; flex-direction: column; gap: var(--space-2);">
-          <%= link_to errors_path(app_context.merge(error_type: @error.error_type)), class: "btn btn-sm" do %>
+          <%= link_to errors_path(app_context.merge(error_type: @error.error_type, unresolved: '0')), class: "btn btn-sm" do %>
             <i class="bi bi-filter"></i> View Similar Errors
           <% end %>
           <% if @error.user_id %>
-            <%= link_to errors_path(app_context.merge(user_id: @error.user_id)), class: "btn btn-sm" do %>
+            <%= link_to errors_path(app_context.merge(user_id: @error.user_id, unresolved: '0')), class: "btn btn-sm" do %>
               <i class="bi bi-person"></i> View User's Errors
             <% end %>
           <% end %>
-          <%= link_to errors_path(app_context.merge(platform: @error.platform)), class: "btn btn-sm" do %>
+          <%= link_to errors_path(app_context.merge(platform: @error.platform, unresolved: '0')), class: "btn btn-sm" do %>
             <i class="bi bi-phone"></i> View <%= @error.platform %> Errors
           <% end %>
           <%= link_to analytics_errors_path(app_context), class: "btn btn-sm" do %>

data/db/migrate/20260503000001_backfill_resolved_status.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+# Backfill `status` column for errors that were bulk-resolved before v0.6.3.
+#
+# Versions 0.6.0 through 0.6.2 had a bug in BatchResolveErrors that set
+# `resolved: true` and `resolved_at` but skipped the `status` column. The
+# errors-index "Resolved" filter pill queries `where(status: 'resolved')`,
+# so bulk-resolved errors silently disappeared from that view even though
+# they were marked resolved.
+#
+# This migration is idempotent: it only updates rows that are out-of-sync
+# (resolved but not status='resolved'). Running it twice is a no-op.
+#
+# See: https://github.com/AnjanJ/rails_error_dashboard
+class BackfillResolvedStatus < ActiveRecord::Migration[7.0]
+  def up
+    return unless table_exists?(:rails_error_dashboard_error_logs)
+    return unless column_exists?(:rails_error_dashboard_error_logs, :status)
+    return unless column_exists?(:rails_error_dashboard_error_logs, :resolved)
+
+    # Use ActiveRecord update_all so the count is portable across adapters
+    # (PostgreSQL, MySQL, SQLite all return the affected row count).
+    table = ActiveRecord::Base.connection.quote_table_name("rails_error_dashboard_error_logs")
+    klass = Class.new(ActiveRecord::Base) { self.table_name = "rails_error_dashboard_error_logs" }
+    updated = klass.where(resolved: true).where("status IS NULL OR status != ?", "resolved")
+                   .update_all(status: "resolved")
+
+    say "Backfilled status='resolved' on #{updated} error log(s) that were bulk-resolved on v0.6.0–v0.6.2."
+  end
+
+  def down
+    # No-op: we cannot reliably distinguish errors that were bulk-resolved
+    # before the fix from errors that are legitimately resolved now. Leaving
+    # status='resolved' is the safe choice on rollback.
+  end
+end

data/lib/rails_error_dashboard/commands/batch_mute_errors.rb

@@ -4,13 +4,14 @@ module RailsErrorDashboard
   module Commands
     # Command: Mute multiple errors at once
     class BatchMuteErrors
-      def self.call(error_ids, muted_by: nil)
-        new(error_ids, muted_by).call
+      def self.call(error_ids, muted_by: nil, reason: nil)
+        new(error_ids, muted_by, reason).call
       end
 
-      def initialize(error_ids, muted_by = nil)
+      def initialize(error_ids, muted_by = nil, reason = nil)
         @error_ids = Array(error_ids).compact
         @muted_by = muted_by
+        @reason = reason
       end
 
       def call
@@ -27,7 +28,8 @@ module RailsErrorDashboard
             error.update!(
               muted: true,
               muted_at: Time.current,
-              muted_by: @muted_by
+              muted_by: @muted_by,
+              muted_reason: @reason
             )
             muted_count += 1
             muted_errors << error

data/lib/rails_error_dashboard/commands/batch_resolve_errors.rb

@@ -31,7 +31,8 @@ module RailsErrorDashboard
               resolved: true,
               resolved_at: Time.current,
               resolved_by_name: @resolved_by_name,
-              resolution_comment: @resolution_comment
+              resolution_comment: @resolution_comment,
+              status: "resolved"
             )
             resolved_count += 1
             resolved_errors << error

data/lib/rails_error_dashboard/version.rb

@@ -1,3 +1,3 @@
 module RailsErrorDashboard
-  VERSION = "0.6.2"
+  VERSION = "0.6.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_error_dashboard
 version: !ruby/object:Gem::Version
-  version: 0.6.2
+  version: 0.6.3
 platform: ruby
 authors:
 - Anjan Jagirdar
@@ -355,6 +355,7 @@ files:
 - db/migrate/20260323000001_add_muted_to_error_logs.rb
 - db/migrate/20260325000001_fix_swallowed_exceptions_index_for_mysql.rb
 - db/migrate/20260326000001_add_issue_tracking_to_error_logs.rb
+- db/migrate/20260503000001_backfill_resolved_status.rb
 - lib/generators/rails_error_dashboard/install/install_generator.rb
 - lib/generators/rails_error_dashboard/install/templates/README
 - lib/generators/rails_error_dashboard/install/templates/initializer.rb
@@ -497,7 +498,7 @@ metadata:
   funding_uri: https://github.com/sponsors/AnjanJ
 post_install_message: |
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-    RED (Rails Error Dashboard) v0.6.2
+    RED (Rails Error Dashboard) v0.6.3
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
   First install: