From 21d7c3f05cef40055380d65f9a22df4f32426ade Mon Sep 17 00:00:00 2001
From: crbelaus <cristian@crbelaus.com>
Date: Mon, 22 Jul 2024 18:59:30 +0200
Subject: [PATCH 1/3] Show occurrences count

Include the total occurrence count both in the dashboard and in the
error detail page.
---
 lib/error_tracker/web/live/dashboard.ex       | 22 ++++++++++++++-----
 .../web/live/dashboard.html.heex              |  5 +++--
 lib/error_tracker/web/live/show.html.heex     |  2 +-
 3 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/lib/error_tracker/web/live/dashboard.ex b/lib/error_tracker/web/live/dashboard.ex
index d0b06be..6dab08e 100644
--- a/lib/error_tracker/web/live/dashboard.ex
+++ b/lib/error_tracker/web/live/dashboard.ex
@@ -64,14 +64,24 @@ defmodule ErrorTracker.Web.Live.Dashboard do
 
     total_errors = Repo.aggregate(query, :count)
 
-    errors_query =
-      query
-      |> order_by(desc: :last_occurrence_at)
-      |> offset((^page - 1) * @per_page)
-      |> limit(@per_page)
+    errors =
+      Repo.all(
+        from query,
+          order_by: [desc: :last_occurrence_at],
+          offset: (^page - 1) * @per_page,
+          limit: @per_page
+      )
+
+    occurrences =
+      errors
+      |> Ecto.assoc(:occurrences)
+      |> group_by([o], o.error_id)
+      |> select([o], {o.error_id, count(o.id)})
+      |> Repo.all()
 
     assign(socket,
-      errors: Repo.all(errors_query),
+      errors: errors,
+      occurrences: Map.new(occurrences),
       total_pages: (total_errors / @per_page) |> Float.ceil() |> trunc
     )
   end
diff --git a/lib/error_tracker/web/live/dashboard.html.heex b/lib/error_tracker/web/live/dashboard.html.heex
index 35d24f6..057f72a 100644
--- a/lib/error_tracker/web/live/dashboard.html.heex
+++ b/lib/error_tracker/web/live/dashboard.html.heex
@@ -41,7 +41,7 @@
     <thead class="text-xs text-gray-700 uppercase bg-gray-50 dark:bg-gray-600 dark:text-gray-400">
       <tr>
         <th scope="col" class="px-4 pr-2 py-3 w-128">Error</th>
-        <th scope="col" class="px-4 py-3 w-72">Last occurrence</th>
+        <th scope="col" class="px-4 py-3 w-72">Occurrences</th>
         <th scope="col" class="px-4 py-3 w-28">Status</th>
         <th scope="col" class="px-4 py-3 w-32"></th>
       </tr>
@@ -62,7 +62,8 @@
           </p>
         </th>
         <td class="px-4 py-4">
-          <%= format_datetime(error.last_occurrence_at) %>
+          <p>Last: <%= format_datetime(error.last_occurrence_at) %></p>
+          <p>Total: <%= @occurrences[error.id] %></p>
         </td>
         <td class="px-4 py-4">
           <.badge :if={error.status == :resolved} color={:green}>Resolved</.badge>
diff --git a/lib/error_tracker/web/live/show.html.heex b/lib/error_tracker/web/live/show.html.heex
index 1b06e5b..d3309f5 100644
--- a/lib/error_tracker/web/live/show.html.heex
+++ b/lib/error_tracker/web/live/show.html.heex
@@ -63,7 +63,7 @@
   </div>
 
   <div class="px-3 md:pl-0 space-y-8">
-    <.section title="Occurrence">
+    <.section title={"Occurrence (#{Enum.count(@occurrences)} total)"}>
       <form phx-change="occurrence_navigation">
         <select name="occurrence_id" class="text-black w-full">
           <option

From 4a77093078eb64ed914706469afede8f21dd2ce6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=93scar=20de=20Arriba?= <oscar@dearriba.es>
Date: Fri, 26 Jul 2024 16:43:53 +0200
Subject: [PATCH 2/3] Use real counters

---
 lib/error_tracker/web/live/show.ex        | 9 ++++++++-
 lib/error_tracker/web/live/show.html.heex | 6 +++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/lib/error_tracker/web/live/show.ex b/lib/error_tracker/web/live/show.ex
index 564b6a1..29244c4 100644
--- a/lib/error_tracker/web/live/show.ex
+++ b/lib/error_tracker/web/live/show.ex
@@ -104,7 +104,14 @@ defmodule ErrorTracker.Web.Live.Show do
       |> List.flatten()
       |> Enum.reverse()
 
-    assign(socket, :occurrences, occurrences)
+    total_occurrences =
+      socket.assigns.error
+      |> Ecto.assoc(:occurrences)
+      |> Repo.aggregate(:count)
+
+    socket
+    |> assign(:occurrences, occurrences)
+    |> assign(:total_occurrences, total_occurrences)
   end
 
   defp related_occurrences(query, num_results) do
diff --git a/lib/error_tracker/web/live/show.html.heex b/lib/error_tracker/web/live/show.html.heex
index d3309f5..0cfd6f0 100644
--- a/lib/error_tracker/web/live/show.html.heex
+++ b/lib/error_tracker/web/live/show.html.heex
@@ -63,7 +63,7 @@
   </div>
 
   <div class="px-3 md:pl-0 space-y-8">
-    <.section title={"Occurrence (#{Enum.count(@occurrences)} total)"}>
+    <.section title={"Occurrence (#{@total_occurrences} total)"}>
       <form phx-change="occurrence_navigation">
         <select name="occurrence_id" class="text-black w-full">
           <option
@@ -85,6 +85,10 @@
       <pre><%= format_datetime(@error.last_occurrence_at) %></pre>
     </.section>
 
+    <.section title="First seen">
+      <pre><%= format_datetime(@error.inserted_at) %></pre>
+    </.section>
+
     <.section title="Status" title_class="mb-3">
       <.badge :if={@error.status == :resolved} color={:green}>Resolved</.badge>
       <.badge :if={@error.status == :unresolved} color={:red}>Unresolved</.badge>

From ded2e74d6604bb2fb6091720cbbf77591a6fc583 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=93scar=20de=20Arriba?= <oscar@dearriba.es>
Date: Fri, 26 Jul 2024 16:44:06 +0200
Subject: [PATCH 3/3] Get stats only for the current page of results

---
 lib/error_tracker/web/live/dashboard.ex | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/lib/error_tracker/web/live/dashboard.ex b/lib/error_tracker/web/live/dashboard.ex
index 6dab08e..0bff44d 100644
--- a/lib/error_tracker/web/live/dashboard.ex
+++ b/lib/error_tracker/web/live/dashboard.ex
@@ -72,9 +72,12 @@ defmodule ErrorTracker.Web.Live.Dashboard do
           limit: @per_page
       )
 
+    error_ids = Enum.map(errors, & &1.id)
+
     occurrences =
       errors
       |> Ecto.assoc(:occurrences)
+      |> where([o], o.error_id in ^error_ids)
       |> group_by([o], o.error_id)
       |> select([o], {o.error_id, count(o.id)})
       |> Repo.all()