Skip to content

fix: console regressions after #5 — empty traffic, audit names, Quick Go, service name - #6

Merged
chenow9 merged 1 commit into
mainfrom
fix/p2-regression-console-ux
Sep 21, 2026
Merged

chenow9 merged 1 commit into
mainfrom
fix/p2-regression-console-ux

Conversation

@chenow9

@chenow9 chenow9 commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Depends on #5 (P2 console UX, which includes P0 #3 and P1 #4). Branch is fix/p2-regression-console-ux from fix/p2-console-ux.

Summary

Fixes four regressions found after deploying the #5 stack. No protocol/security-model changes. No new P3 work.

F1 — Traffic empty state

观测 → 流量 with no data now uses the shared empty-state: Activity icon + 暂无流量 (compact). If there are no nodes, the CTA is 登记节点. Chart axes stay hidden.

F2 — Audit object column

Object column prefers a node/service display name. If live lookup is gone (e.g. after delete), it recovers the name from audit detail (home-nas linux/amd64 → home-nas). Raw ids stay as tooltip / fallback, never a blank cell.

F3 — Quick Go 「全部服务」

Quick Go has an 全部服务 action (same wording as main nav) that opens /mappings. Searching 全部服务 matches it; 添加服务 stays a separate action.

F4 — Service name validation

Service names use the same character rules as node names. !!!bad shows an inline error and cannot advance; API create/update/import also reject it. 工作室 SSH still works.

Files changed

Fix Files
F1 empty-state.tsx, rate-chart.tsx, traffic-page.tsx
F2 labels.ts, http.go, views.go, labels.test.ts, service_test.go
F3 quick-launcher.tsx, messages.ts, quick-launch.test.ts
F4 service.ts, http.go, mutate.go, transfer.go, service.test.ts, service_test.go

How verified

  • Unit: audit object prefers home-nas over nde_abc; Quick Go query 全部服务; validateService({ name: "!!!bad" }) fails.
  • API: go test ./internal/control/ including delete-then-audit targetName=home-nas and mapping create !!!bad → 400.
  • npx tsc --noEmit and eslint on touched files.

Checklist

  • F1 Traffic empty state matches the shared pattern
  • F2 Audit object column prefers display names
  • F3 Quick Go includes 全部服务
  • F4 Service name !!!bad cannot be saved

…ce name rules

Match traffic empty to the shared empty-state, prefer display names in
audit objects, add 全部服务 to Quick Go, and reject invalid service names
in the console and API.
@chenow9
chenow9 changed the base branch from fix/p2-console-ux to main September 21, 2026 01:26
@chenow9
chenow9 merged commit dd281e9 into main Sep 21, 2026
2 checks passed
@chenow9

chenow9 commented Sep 21, 2026

Copy link
Copy Markdown
Owner Author

合并顺序:#3 → #4 → #5 → #6。

  • P0:访问方式、接入确认、名称/主机/CIDR 校验。
  • P1:设置、空状态/加载、改为公开访问的二次确认。
  • P2:总览首页、统一导航、审计人话、浅色深色同一主色、禁用操作说明原因。
  • F1–F4:流量空状态、审计对象列优先显示名称、快速前往「全部服务」、服务名称与节点同规则。

未改协议与安全模型。云主机针对性回归 F1–F4 已通过。

@chenow9
chenow9 deleted the fix/p2-regression-console-ux branch September 29, 2026 07:42
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