exception_hunter 0.2.0 → 1.0.0

data/app/presenters/exception_hunter/error_group_presenter.rb

@@ -0,0 +1,25 @@
+module ExceptionHunter
+  class ErrorGroupPresenter
+    delegate_missing_to :error_group
+
+    def initialize(error_group)
+      @error_group = error_group
+    end
+
+    def self.wrap_collection(collection)
+      collection.map { |error_group| new(error_group) }
+    end
+
+    def self.format_occurrence_day(day)
+      day.to_date.strftime('%A, %B %d')
+    end
+
+    def show_for_day?(day)
+      last_occurrence.in_time_zone.to_date == day.to_date
+    end
+
+    private
+
+    attr_reader :error_group
+  end
+end

data/app/presenters/exception_hunter/error_presenter.rb

@@ -1,6 +1,7 @@
 module ExceptionHunter
   class ErrorPresenter
     delegate_missing_to :error
+    delegate :tags, to: :error_group
 
     BacktraceLine = Struct.new(:path, :file_name, :line_number, :method_call)
 
@@ -9,13 +10,13 @@ module ExceptionHunter
     end
 
     def backtrace
-      error.backtrace.map do |line|
+      (error.backtrace || []).map do |line|
         format_backtrace_line(line)
       end
     end
 
     def environment_data
-      error.environment_data.except('params')
+      error.environment_data&.except('params') || {}
     end
 
     def tracked_params

data/app/views/exception_hunter/devise/sessions/new.html.erb

@@ -0,0 +1,24 @@
+<div class="login_form_container">
+  <div class="login_left_container">
+    <div class="left_column">Exception Hunter</div>
+  </div>
+  <div class="login_right_container">
+    <%= form_for(resource, as: resource_name, url: exception_hunter_create_session_path) do |f| %>
+    <div class="row">
+      Sign into your account
+    </div>
+    <div class="login_row row"></div>
+    <div class="login_row row">
+      <%= f.email_field :email, autofocus: true, autocomplete: "email", placeholder: 'Email' %>
+    </div>
+    <div class="login_row row">
+      <%= f.password_field :password, autocomplete: "current-password", placeholder: 'Password'%>
+    </div>
+    <div class="login_button row">
+      <div class="column column-50 column-offset-50">
+          <input class="button-log-in" type="submit" value="Log in">
+      </div>
+    </div>
+    <% end %>
+  </div>
+</div>

data/app/views/exception_hunter/errors/_error_row.erb

@@ -0,0 +1,52 @@
+<div class="row error-row">
+  <div class="column column-10 error-cell">
+    <% error.tags.each do |tag| %>
+      <div class="error-cell__tags">
+        <span class="error-tag"><%= tag %></span>
+      </div>
+    <% end %>
+  </div>
+  <div class="column column-33 error-cell error-cell__message">
+    <%= link_to error.message, error_path(error.id), class: %w[error-message] %>
+  </div>
+
+  <div class="column column-15 error-cell">
+    <% if error.first_occurrence.present? %>
+      <%= time_ago_in_words(error.first_occurrence) %> ago
+    <% else %>
+      Never
+    <% end %>
+  </div>
+
+  <div class="column column-15 error-cell">
+    <% if error.last_occurrence.present? %>
+      <%= time_ago_in_words(error.last_occurrence) %> ago
+    <% else %>
+      Never
+    <% end %>
+  </div>
+
+  <div class="column column-8 error-cell">
+    <%= error.total_occurrences %>
+  </div>
+
+   <div class="column column-18 error-cell">
+    <div class="row">
+      <% if error.active? %>
+        <div class="column mr-5">
+        <%= display_action_button('resolve', error) %>
+        </div>
+        <div class="column">
+          <%= display_action_button('ignore', error) %>
+        </div>
+      <% elsif error.ignored? %>
+        <div class="column">
+          <%= button_to('Reopen', reopen_path(error_group: { id: error.id }),
+                      class: %w[button button-outline resolve-button],
+                      data: { confirm: 'Are you sure you want to reopen this error?' }) %>
+        </div>
+      <% end %>
+    </div>
+   </div>
+</div>
+

data/app/views/exception_hunter/errors/_error_summary.erb

@@ -1,9 +1,9 @@
 <% if error.environment_data.empty? %>
-  <div class="row error-row-no-border data-title">
+  <div class="row error-row data-title">
     Unfortunately, no environment information has been registered for this error.
   </div>
 <% else %>
-  <div class="row error-row-no-border data-title">
+  <div class="row error-row data-title">
     Environment Data
   </div>
   <pre class="tracked-data">
@@ -13,7 +13,7 @@
 <% end %>
 
 <% unless error.tracked_params.nil? %>
-  <div class="row error-row-no-border data-title">
+  <div class="row error-row data-title">
     Tracked Params
   </div>
   <pre class="tracked-data">
@@ -23,11 +23,11 @@
 <% end %>
 
 <% if error.custom_data.nil? %>
-  <div class="row error-row-no-border data-title">
+  <div class="row error-row data-title">
     No custom data included.
   </div>
 <% else %>
-  <div class="row error-row-no-border data-title">
+  <div class="row error-row data-title">
     Custom Data
   </div>
   <pre class="tracked-data">

data/app/views/exception_hunter/errors/_errors_table.erb

@@ -0,0 +1 @@
+<%= render partial: 'exception_hunter/errors/error_row', collection: errors, as: :error %>

data/app/views/exception_hunter/errors/_last_7_days_errors_table.erb

@@ -0,0 +1,12 @@
+<% today_errors = errors.select { |error| error.show_for_day?(Date.current) } %>
+<%= render partial: 'exception_hunter/errors/error_row', collection: today_errors, as: :error %>
+
+<% yesterday_errors = errors.select { |error| error.show_for_day?(Date.yesterday) } %>
+<div class="errors-date-group">Yesterday</div>
+<%= render partial: 'exception_hunter/errors/error_row', collection: yesterday_errors, as: :error %>
+
+<% (2..6).each do |i| %>
+  <% errors_on_day = errors.select { |error| error.show_for_day?(i.days.ago) } %>
+  <div class="errors-date-group"><%= ExceptionHunter::ErrorGroupPresenter.format_occurrence_day(i.days.ago) %></div>
+  <%= render partial: 'exception_hunter/errors/error_row', collection: errors_on_day, as: :error %>
+<% end %>

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

@@ -1,36 +1,91 @@
-<div class="row statistics-row">
-  <div class="column column-offset-50 column-25">
-    <div class="statistics__cell">
-      <%= number_with_delimiter(@errors_count) %> Total errors
+<div class="row">
+  <div class="column column-80">
+    <div class="errors-tabs">
+      <div class="errors-tab <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::LAST_7_DAYS_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::LAST_7_DAYS_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::LAST_7_DAYS_TAB) %>
+            </div>
+            <div class="errors-tab__description">
+              Errors in the last 7 days
+            </div>
+          </div>
+        <% end %>
+      </div>
+
+      <div class="errors-tab <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::CURRENT_MONTH_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::CURRENT_MONTH_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::CURRENT_MONTH_TAB) %>
+            </div>
+            <div class="errors-tab__description">
+              Errors this month
+            </div>
+          </div>
+        <% end %>
+      </div>
+
+      <div class="errors-tab <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::TOTAL_ERRORS_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::TOTAL_ERRORS_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::TOTAL_ERRORS_TAB) %>
+            </div>
+            <div class="errors-tab__description">
+              Total errors
+            </div>
+          </div>
+        <% end %>
+      </div>
+
+      <div class="errors-tab errors-tab__resolved <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::RESOLVED_ERRORS_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::RESOLVED_ERRORS_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::RESOLVED_ERRORS_TAB) || '-' %>
+            </div>
+            <div class="errors-tab__description">
+              Resolved
+            </div>
+          </div>
+        <% end %>
+      </div>
+
+      <div class="errors-tab errors-tab__ignored <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::IGNORED_ERRORS_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::IGNORED_ERRORS_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::IGNORED_ERRORS_TAB) || '-' %>
+            </div>
+            <div class="errors-tab__description">
+              Ignored
+            </div>
+          </div>
+        <% end %>
+      </div>
     </div>
   </div>
-  <div class="column column-25">
-    <div class="statistics__cell">
-      <%= number_with_delimiter(@month_errors) %> Errors this month
-    </div>
+
+  <div class="column column-10 column-offset-10">
+    <%= button_to 'Purge', purge_errors_path,
+                  class: %w[button purge-button],
+                  method: :delete,
+                  data: { confirm: 'This will delete all stale errors, do you want to continue?' } %>
   </div>
 </div>
 
-<div class="row error-row error-row--header">
-  <div class="column column-75">Message</div>
-  <div class="column column-15">Last Occurrence</div>
-  <div class="column column-10">Occurrences</div>
-</div>
+<div class="errors__container">
+  <div class="row error-row error-row--header">
+    <div class="column column-10">Tags</div>
+    <div class="column column-33">Message</div>
+    <div class="column column-15">First Occurrence</div>
+    <div class="column column-15">Last Occurrence</div>
+    <div class="column column-8">Total</div>
+    <div class="column column-18"></div>
 
-<% @errors.each do |error| %>
-  <div class="row error-row">
-    <div class="column column-75 error-cell">
-      <%= link_to error.message, error_path(error.id) %>
-    </div>
-    <div class="column column-15 error-cell error-cell--highlight">
-      <% if error.last_occurrence.present? %>
-        <%= time_ago_in_words(error.last_occurrence) %> ago
-      <% else %>
-        Never
-      <% end %>
-    </div>
-    <div class="column column-10 error-cell error-cell--highlight">
-      <%= error.total_occurrences %>
-    </div>
   </div>
-<% end %>
+
+  <%= render partial: @dashboard.partial_for_tab, locals: { errors: @errors } %>
+</div>

data/app/views/exception_hunter/errors/pagy/_pagy_nav.html.erb

@@ -1,17 +1,17 @@
-<% link = pagy_link_proc(pagy) %>
-<nav aria-label="pager" class="pagy_nav pagination" role="navigation">
-  <% if pagy.prev %>
-  <span class="page prev"><%== link.call(pagy.prev, '◄', 'aria-label="previous"') %></span>
-  <% else %>
-    <span class="page prev disabled">
-      ◄
-    </span>
+<div class="error-occurrences__nav">
+  <%= link_to pagy_url_for(1, pagy) do %>
+    <%= button_tag 'First', class: %w[button button-outline error-occurrences__nav-link], disabled: pagy.prev.nil? %>
   <% end %>
-  <% if pagy.next %>
-    <span class="page next"><%== link.call(pagy.next, '►', 'aria-label="next"') %></span>
-  <% else %>
-    <span class="page next disabled">
-      ►
-    </span>
+  <%= link_to pagy_url_for(pagy.prev, pagy) do %>
+    <%= button_tag '<', class: %w[button button-outline error-occurrences__nav-link], disabled: pagy.prev.nil? %>
   <% end %>
-</nav>
+  <div class="error-occurrences__nav-current">
+    <%= occurred_at %> (<%= pagy.page %>/<%= pagy.last %>)
+  </div>
+  <%= link_to pagy_url_for(pagy.next, pagy) do %>
+    <%= button_tag '>', class: %w[button button-outline error-occurrences__nav-link], disabled: pagy.next.nil? %>
+  <% end %>
+  <%= link_to pagy_url_for(pagy.last, pagy) do %>
+    <%= button_tag 'Last', class: %w[button button-outline error-occurrences__nav-link], disabled: pagy.next.nil? %>
+  <% end %>
+</div>

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

@@ -1,33 +1,69 @@
-<div class="row">
-  <div class="column column-offset-80 column-20">
-    <%= render partial: 'exception_hunter/errors/pagy/pagy_nav', locals: { pagy: @pagy } %>
+<div class="row error-occurrence__header">
+  <div class="column column-10">
+    <% @error.tags.each do|tag| %>
+    <div class="error-cell__tags">
+      <span class="error-tag"><%= tag %></span>
+    </div>
+    <% end %>
+  </div>
+  <div class="column column-80">
+    <div class="error-title">
+      <%= @error.class_name %>: <%= @error.message %>
+    </div>
+  </div>
+  <div class="column column-10">
+    <%= button_to('Resolve', resolved_errors_path(error_group: { id: @error.error_group_id }),
+                  method: :post,
+                  class: %w[button resolve-button],
+                  data: { confirm: 'Are you sure you want to resolve this error?' }) %>
   </div>
 </div>
 
-<div class="error-title">
-  <%= @error.class_name %>: <%= @error.message %>
-</div>
-<div class="error-occurred_at">
-  <%= @error.occurred_at %>
+<div class="row">
+  <div class="column column-50">
+    <ul data-tabs class="errors-tabs">
+      <li class="errors-tab">
+        <a data-tabby-default href="#summary">
+          <div class="errors-tab__content">
+            Summary
+          </div>
+        </a>
+      </li>
+      <li class="errors-tab">
+        <a href="#backtrace">
+          <div class="errors-tab__content">
+            Backtrace
+          </div>
+        </a>
+      </li>
+      <li class="errors-tab">
+        <a href="#user-data">
+          <div class="errors-tab__content">
+            User Data
+          </div>
+        </a>
+      </li>
+    </ul>
+  </div>
+  <div class="column column-50">
+    <%= render partial: 'exception_hunter/errors/pagy/pagy_nav', locals: { pagy: @pagy, occurred_at: @error.occurred_at } %>
+  </div>
 </div>
 
-<ul data-tabs>
-  <li><a data-tabby-default href="#summary">Summary</a></li>
-  <li><a href="#backtrace">Backtrace</a></li>
-  <li><a href="#user-data">User Data</a></li>
-</ul>
 
-<div class="tab-content">
-  <div id="summary">
-    <%= render partial: 'exception_hunter/errors/error_summary', locals: { error: @error } %>
-  </div>
+<div class="errors__container">
+  <div class="tab-content">
+    <div id="summary">
+      <%= render partial: 'exception_hunter/errors/error_summary', locals: { error: @error } %>
+    </div>
 
-  <div id="backtrace">
-    <%= render partial: 'exception_hunter/errors/error_backtrace', locals: { error: @error } %>
-  </div>
+    <div id="backtrace">
+      <%= render partial: 'exception_hunter/errors/error_backtrace', locals: { error: @error } %>
+    </div>
 
-  <div id="user-data">
-    <%= render partial: 'exception_hunter/errors/error_user_data', locals: { error: @error } %>
+    <div id="user-data">
+      <%= render partial: 'exception_hunter/errors/error_user_data', locals: { error: @error } %>
+    </div>
   </div>
 </div>
 

data/app/views/layouts/exception_hunter/application.html.erb

@@ -7,12 +7,11 @@
 
   <%= favicon_link_tag 'exception_hunter/logo.png' %>
 
-  <link rel="stylesheet" href="//fonts.googleapis.com/css?family=Roboto:300,300italic,700,700italic">
+  <link rel="stylesheet" href="https://rsms.me/inter/inter.css">
   <link rel="stylesheet" href="//cdnjs.cloudflare.com/ajax/libs/normalize/5.0.0/normalize.css">
   <link rel="stylesheet" href="//cdnjs.cloudflare.com/ajax/libs/milligram/1.3.0/milligram.css">
 
   <!-- Get patch fixes within a minor version -->
-  <link rel="stylesheet" type="text/css" href="https://cdn.jsdelivr.net/gh/cferdinandi/tabby@12.0/dist/css/tabby-ui.min.css">
   <script src="https://cdn.jsdelivr.net/gh/cferdinandi/tabby@12.0/dist/js/tabby.polyfills.min.js"></script>
 
   <%= stylesheet_link_tag "exception_hunter/application", media: "all" %>
@@ -22,18 +21,80 @@
 <div class="wrapper">
   <nav class="nav">
     <div class="container container__nav">
-      <div class="nav__logo">
-        <%= link_to errors_path do %>
-          <%= image_tag 'exception_hunter/logo.png', alt: 'Logo' %>
-        <% end %>
+      <div class="row">
+        <div class="column column-40">
+          <%= link_to errors_path do %>
+            <div class="nav__title">
+              Exception Hunter / <%= application_name %>
+            </div>
+          <% end %>
+        </div>
+        <div class="column column-10 column-offset-50">
+          <div class="logout">
+            <% if current_admin_user? %>
+              <div class="sign_out">
+                <%= link_to exception_hunter_logout_path do %>
+                  Sign out
+                <% end %>
+              </div>
+            <% end %>
+          </div>
+        </div>
       </div>
     </div>
   </nav>
 
   <div class="container">
+    <% if flash[:notice] %>
+      <div id="flash-notice" class="flash flash--notice row">
+        <div class="column column-80">
+          <%= flash[:notice] %>
+        </div>
+        <div class="column column-20">
+          <div class="button button-clear button-dismiss" data-dismiss="#flash-notice">
+            <%= ['Cool!', 'Nice!', 'Ok', 'Dismiss', 'Fine. Whatever.', 'I know that'].sample %>
+          </div>
+        </div>
+      </div>
+    <% end %>
+
     <%= yield %>
   </div>
+
+  <div class="footer">
+    Made with ♥ at <a href="https://www.rootstrap.com/" class="text--underline">Rootstrap</a>
+  </div>
 </div>
 
+<script type="text/javascript" charset="utf-8">
+  function confirmable(element) {
+    const message = element.getAttribute('data-confirm')
+    element.form.addEventListener('submit', confirm(message))
+
+    function confirm(message) {
+      return function (event) {
+        if (!window.confirm(message)) {
+          event.preventDefault()
+        }
+      }
+    }
+  }
+
+  function dismissible(element) {
+    element.addEventListener('click', dismiss(element))
+
+    function dismiss(element) {
+      return function () {
+        const selector = element.getAttribute('data-dismiss')
+        const elementToDismiss = document.querySelector(selector)
+        elementToDismiss.remove()
+      }
+    }
+  }
+
+  Array.from(document.querySelectorAll('[data-confirm]')).forEach(confirmable)
+  Array.from(document.querySelectorAll('[data-dismiss]')).forEach(dismissible)
+</script>
+
 </body>
 </html>