From 764ca9ae86abb08ace08fb8106e90b7f481ae573 Mon Sep 17 00:00:00 2001
From: Tim DiLauro <tdilauro@users.noreply.github.com>
Date: Tue, 1 Oct 2024 21:55:42 -0400
Subject: [PATCH] Re-remove Total Patrons from the patron stats group. (#138)

---
 src/components/LibraryStats.tsx                |  1 -
 src/components/StatsPatronGroup.tsx            | 11 ++++++-----
 src/components/__tests__/LibraryStats-test.tsx |  6 ++----
 3 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/components/LibraryStats.tsx b/src/components/LibraryStats.tsx
index 62e091bb7..5cdea404c 100644
--- a/src/components/LibraryStats.tsx
+++ b/src/components/LibraryStats.tsx
@@ -65,7 +65,6 @@ const LibraryStats = ({ stats, library }: LibraryStatsProps) => {
       <ul className={`stats ${statsLayoutClass}`}>
         <li className="stat-group stat-patrons-group">
           <StatsPatronGroup
-            total={patrons.total}
             withActiveLoan={patrons.withActiveLoan}
             withActiveLoanOrHold={patrons.withActiveLoanOrHold}
             heading="Current Circulation Activity"
diff --git a/src/components/StatsPatronGroup.tsx b/src/components/StatsPatronGroup.tsx
index 3aa61e60a..613e96ec1 100644
--- a/src/components/StatsPatronGroup.tsx
+++ b/src/components/StatsPatronGroup.tsx
@@ -25,11 +25,12 @@ const StatsPatronGroup = ({
             label="Total Patrons"
             value={total}
             tooltip={`
-              Total number of patrons in the Palace System.
-              Please note:
-              this number could be artificially inflated if you have an
-              Aspen integration. We are working to address the issue and
-              will update you when it’s resolved.
+              Total number of patron records currently cached in the Palace System.
+
+              Please note: This value does not necessarily reflect direct patron
+              interaction with the Palace applications. Use of external discovery
+              systems that interact with Palace on a patron's behalf may inflate
+              this number.
               `
               .replace(/(?:\s|\r\n|\r|\n)+/g, " ")
               .trim()}
diff --git a/src/components/__tests__/LibraryStats-test.tsx b/src/components/__tests__/LibraryStats-test.tsx
index b2051a322..9027ea764 100644
--- a/src/components/__tests__/LibraryStats-test.tsx
+++ b/src/components/__tests__/LibraryStats-test.tsx
@@ -106,14 +106,12 @@ describe("LibraryStats", () => {
       /* Patrons */
       expect(groups.at(0).text()).to.contain("Current Circulation Activity");
       statItems = groups.at(0).find("SingleStatListItem");
-      expectStats(statItems.at(0).props(), "Total Patrons", 132);
-      expectStats(statItems.at(1).props(), "Patrons With Active Loans", 21);
+      expectStats(statItems.at(0).props(), "Patrons With Active Loans", 21);
       expectStats(
-        statItems.at(2).props(),
+        statItems.at(1).props(),
         "Patrons With Active Loans or Holds",
         23
       );
-      expect(groups.at(0).text()).to.contain("132Total Patrons");
       expect(groups.at(0).text()).to.contain("21Patrons With Active Loans");
       expect(groups.at(0).text()).to.contain(
         "23Patrons With Active Loans or Holds"