style(channels): simplify channel status indicators#862
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9550ec1eff
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| <div className="flex items-center gap-2"> | ||
| <p className="text-[13px] font-medium text-foreground truncate">{displayName}</p> | ||
| <Badge className={cn('rounded-full border px-2 py-0.5 text-[10px] font-medium', statusTone(account.status))}> | ||
| {statusLabel(account.status)} | ||
| </Badge> | ||
| </div> |
There was a problem hiding this comment.
Restore per-account connection status in channel rows
This change removes the only explicit status indicator for each account, so accounts that are connecting/disconnected now render identically to healthy ones unless they also have lastError or a degraded reason. That is a functional regression when a channel has multiple accounts, because the group-level indicator is aggregated (pickChannelRuntimeStatus returns connected as soon as any account is connected), so a partially broken channel can appear healthy with no per-account signal to identify which account needs attention.
Useful? React with 👍 / 👎.
…overy (ValueCell-ai#855 ValueCell-ai#862 ValueCell-ai#867 ValueCell-ai#892 ValueCell-ai#841) Channel health banner, diagnostics snapshot, gateway restart button, status indicator styling, multi-account binding fix.
No description provided.