Metrics page fixes

[simo6529]

Summary by CodeRabbit

• New Features

  • Floating heart link on the home page that opens the network Health dashboard

• Updates

  • Renamed "Metrics" to "Health" across navigation and page titles; updated metric labels (e.g., Posters, Posts, Active Votes, Identities)
  • Network Stats links updated to point to Health
  • Chart display order adjusted; added subtle heartbeat animation styling
  • Removed Discord page/contact links from site

• Tests

  • Added and adjusted tests for the new heart link and Health dashboard navigation; minor test assertion refinements

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR renames network routes/labels from "Metrics" to "Health", updates related navigation links, removes Discord content, adds a floating heart link in the hero that points to the health dashboard (with Tailwind animation), and updates/adds tests accordingly.

Changes

Cohort / File(s)	Summary
Navigation Route Migration app/network/definitions/page.client.tsx, app/network/tdh/page.client.tsx, app/network/tdh/historic-boosts/page.client.tsx, components/header/AppSidebar.tsx, hooks/useSidebarSections.ts	Repointed navigation links and menu entries from /network/metrics and /network/metrics/network-tdh → /network/health and /network/health/network-tdh; label "Metrics" → "Health" where applicable.
Page Content & Metadata Updates app/network/health/page.tsx, app/network/health/page.client.tsx, app/network/health/components/MintMetricsCard.tsx	Changed page metadata/title from "Metrics" → "Health"; renamed metric card labels (e.g., Distinct Droppers→Posters, Drops Created→Posts, Voting Power→Active Votes, Profile Count→Identities); MintMetricsCard now passes reversed data via data.toReversed().
Discord Content Removal app/om/discord/page.tsx, components/about/AboutContactUs.tsx, components/footer/Footer.tsx	Removed the Discord page; removed Discord contact block from AboutContactUs; removed Discord link/image from Footer.
Hero Header Enhancement components/home/hero/HeroHeader.tsx, __tests__/components/home/hero/HeroHeader.test.tsx, tests/home/home.spec.ts, tailwind.config.ts	Added a floating heart Link in the hero that navigates to /network/health; added heart-beat-soft keyframes/animation; added unit and e2e tests validating the hero link and heading.
Tests & Formatting Updates __tests__/app/omPages.test.tsx, tests/pages/network-tdh.spec.ts	Adjusted test string quoting/formatting; replaced a role-based Discord link assertion with text-based assertion; updated beforeEach navigation path in TDH tests to /network/health/network-tdh.
Minor UI / Signature Tweaks components/user/user-page-header/about/UserPageHeaderAboutEditError.tsx	Reordered Tailwind classes, removed an inline paragraph from error display, and added a trailing comma in the destructured props signature (formatting change).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• xTDH info page #1637: Edits same hooks/useSidebarSections.ts (sidebar/network entries) — likely overlapping navigation changes.
• More <a> to next/link #1507: Modifies components/about/AboutContactUs.tsx (Discord/link handling) — directly related to Discord removals.
• Stats to metrics #1744: Updates AppSidebar/useSidebarSections/network routing — overlaps on Network Stats/TDH link changes.

Suggested reviewers

• prxt6529
• ragnep

Poem

🐰 I hopped and found the nav anew,
Metrics turned to Health, a brighter view,
A tiny heart now points the way,
It beats soft, then hops away,
Routes refreshed — a carrot-cheer hooray!

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The PR title 'Metrics page fixes' is vague and generic, failing to describe the actual primary changes which involve renaming 'Metrics' to 'Health' throughout the codebase and adding health dashboard navigation.	Consider a more specific title that reflects the main changes, such as 'Rename Metrics to Health and add network health dashboard link' or 'Rebrand network metrics page to health dashboard'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch metrics-page-fixes

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

app/network/health/page.tsx (1)

1-6: Component names still reference "Metrics" after the rename to "Health".

MetricsPageClient (import) and MetricsPage (exported default) retain the old naming while everything else in this PR has moved to "Health". Consider renaming for consistency.

Proposed fix

-import MetricsPageClient from "./page.client";
+import HealthPageClient from "./page.client";
 import { getAppMetadata } from "@/components/providers/metadata";
 import type { Metadata } from "next";

-export default function MetricsPage() {
-  return <MetricsPageClient />;
+export default function HealthPage() {
+  return <HealthPageClient />;
 }

This also requires renaming the default export in ./page.client.tsx to HealthPageClient.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

components/about/AboutContactUs.tsx (1)

58-83: ⚠️ Potential issue | 🟠 Major

Duplicate list item after Discord removal — likely a leftover.

The second <li> (Lines 58–83) lists the exact same Twitter handles as the first <li> (Lines 30–56), but without the "on Twitter" qualifier. This appears to be a remnant of the Discord block removal — previously this bullet likely directed users to contact via Discord. Consider removing this duplicate entry entirely.

Proposed fix

             on Twitter
             </li>
-            <br />
-            <li>
-              Trying to get the attention of &#64;
-              <Link
-                href="https://x.com/punk6529"
-                target="_blank"
-                rel="noopener noreferrer"
-              >
-                punk6529
-              </Link>
-              , &#64;
-              <Link
-                href="https://x.com/6529er"
-                target="_blank"
-                rel="noopener noreferrer"
-              >
-                6529er
-              </Link>{" "}
-              or &#64;
-              <Link
-                href="https://x.com/teexels"
-                target="_blank"
-                rel="noopener noreferrer"
-              >
-                teexels
-              </Link>{" "}
-            </li>

🤖 Fix all issues with AI agents

In `@app/network/health/page.tsx`:
- Line 10: The page metadata description is inconsistent with the updated title;
update the getAppMetadata call in page.tsx so the description matches the new
title by changing the description string from "Metrics dashboard" to "Health
dashboard" (in the return getAppMetadata({ title: "Health", description: ... })
call).

🧹 Nitpick comments (3)

app/network/health/page.tsx (1)

1-6: Component and import names still reference "Metrics".

MetricsPage (Line 5) and MetricsPageClient (Line 1) haven't been renamed to match the "Health" branding. This is purely internal naming, so it's low priority, but worth aligning if you're doing a full rename pass.

hooks/useSidebarSections.ts (1)

33-38: Subsection still named "Metrics" while its child is now "Health" — intentional?

The subsection label on line 33 remains "Metrics" but the first item was renamed to "Health" (line 35). If the intent is to rename the concept from "Metrics" to "Health", consider renaming the subsection too for consistency. If the subsection deliberately groups multiple metric-related pages under a broader "Metrics" heading, this is fine as-is.

app/network/health/page.client.tsx (1)

28-29: Component still named MetricsPageClient while the page is now "Health".

Consider renaming to HealthPageClient (and the file if applicable) for consistency with the new page title. Not urgent, but reduces cognitive friction.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
1 Accepted issue

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud