Skip to content

fix(ui): route dashboard View Hosts CTAs to /environment - #127

Open
greg12579 wants to merge 1 commit into
SiriusScan:mainfrom
greg12579:fix/dashboard-host-link
Open

greg12579 wants to merge 1 commit into
SiriusScan:mainfrom
greg12579:fix/dashboard-host-link

Conversation

@greg12579

Copy link
Copy Markdown

Summary

The dashboard had two "View Hosts" CTAs linking to /host, which 404s. The host-list page lives at /environment — the /host/ route is only defined as pages/host/[ip].tsx (the dynamic detail page), with no index.

Fixes the CTAs in two places:

  • src/pages/dashboard.tsx:181 — stats-card CTA href
  • src/pages/dashboard.tsx:324 — ActionButton router.push

Both now route to /environment.

Test plan

  • On the dashboard, click the View Hosts link on the host stats card → lands on /environment, no 404
  • On the dashboard, click the View All Hosts quick-action button → lands on /environment, no 404
  • Regression: /environment still renders the host table normally

🤖 Generated with Claude Code

The dashboard had two "View Hosts" CTAs (stats card and quick-action
button) linking to /host, which 404s. The host-list page lives at
/environment; /host is reserved for the dynamic /host/[ip] detail route
and has no index page.

- src/pages/dashboard.tsx:181 stats-card CTA
- src/pages/dashboard.tsx:324 ActionButton onClick
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant