✨ Display active user count in navbar header#3904
Conversation
|
👋 Hey @Copilot — thanks for opening this PR!
This is an automated message. |
❌ PR Title Verification FailedYour PR title does not follow the required format. Current title: Required FormatPR titles must start with one of these emoji prefixes:
How to FixEdit your PR title to start with the appropriate emoji. For example:
You can edit the title by clicking the Edit button next to your PR title. This comment was automatically posted by the PR Title Verifier workflow. |
✅ Deploy Preview for kubestellarconsole ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Fixes #3416 Signed-off-by: GitHub Copilot <copilot@github.com> Agent-Logs-Url: https://github.com/kubestellar/console/sessions/ec338a26-9145-4df5-9c0e-6ad1c1487ae9 Co-authored-by: clubanderson <407614+clubanderson@users.noreply.github.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits. 📝 Please follow instructions in the contributing guide to update your commits with the DCO Full details of the Developer Certificate of Origin can be found at developercertificate.org. The list of commits missing DCO signoff:
DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
There was a problem hiding this comment.
Pull request overview
Renders the existing ActiveUsersWidget in the main navigation so the active user count is visible in both desktop (lg+) and mobile layouts.
Changes:
- Add
ActiveUsersWidgetto the navbar’s lg+ desktop widget row. - Add
ActiveUsersWidgetto the mobile overflow (“more”) menu items.
|
Thank you for your contribution! Your PR has been merged. Check out what's new:
Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey |
ActiveUsersWidgetexisted with full WebSocket presence tracking, polling, demo mode support, and error handling — but was never rendered anywhere in the UI.Changes Made
ActiveUsersWidgetinNavbar.tsxalongsideTokenUsageWidgetin the lg+ desktop item rowActiveUsersWidgetto the mobile overflow menu for full responsive coverageChanges Made
ActiveUsersWidgetto navbar desktop (lg+) itemsActiveUsersWidgetto mobile overflow menuChecklist
git commit -s)Screenshots or Logs (if applicable)
ActiveUsersWidgetnow appears in the navbar betweenTokenUsageWidgetandFeatureRequestButtonat lg+ breakpoints, and in the mobile overflow sheet at narrower widths.👀 Reviewer Notes
Zero logic changes — purely a rendering omission fix. The widget, hook, backend endpoint (
/api/active-users), and i18n strings were all already in place.🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.