feat: refine proxy group member controls#52
Conversation
Split advanced proxy group members into node and rule group sections, with separate clear actions for enabled nodes and enabled rule groups. Update proxy group summaries to show routing rules, nodes, and rule sets as separate counts.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c85f282984
ℹ️ 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".
|
Thanks for this, @AAEE86 — appreciate you digging into the proxy group summaries. Your PR surfaced something we'd missed: the member count on each group card was inflated, because it counted every entry (including We're going to hold off on the rest, for two reasons:
So we'll carry the accurate node count forward and close this out here. Thanks again for catching the counting issue — it's a real improvement. 🙏 |
Split advanced proxy group members into node and rule group sections, with separate clear actions for enabled nodes and enabled rule groups.
Update proxy group summaries to show routing rules, nodes, and rule sets as separate counts.