Skip to content

feat: add an Online view and column to the Network Users list - #336

Open
josephfusco wants to merge 1 commit into
feature/network-sites-columnfrom
feature/network-users-list
Open

feat: add an Online view and column to the Network Users list#336
josephfusco wants to merge 1 commit into
feature/network-sites-columnfrom
feature/network-users-list

Conversation

@josephfusco

@josephfusco josephfusco commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Part of the network presence stack, merged bottom-up:

  1. feat: add a network-wide presence summary table #332 summary table
  2. feat: push each site's online set into the network summary #333 write path
  3. feat: read the network summary as a capped snapshot #334 read path
  4. feat: add an Online column to the Network Sites list #335 Sites list column
  5. feat: add an Online view and column to the Network Users list #336 Users list (this one)
  6. feat: add a Who's Online widget to the Network Admin dashboard #337 network dashboard widget

The summary is keyed by site and this screen is keyed by user.

Part of #298
Closes #322

Use of AI Tools

AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Sonnet 5, Claude Opus 5
Used for: Assisting with design, implementation, and testing

@github-actions

Copy link
Copy Markdown
Contributor

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props joefusco.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

▶ Preview in WordPress Playground

Open in WordPress Playground

Boots a fresh WordPress with this PR's presence-api build, seeds 5 demo users, and drops you on the dashboard.

Stress-test variant: 40 demo users · Built from 8cee7a26266fac8e99a2f2d17612dbb20dfd1482. Auto-updates when you push.

@josephfusco

josephfusco commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator Author

#322 is closed by this one; test_the_network_online_view_is_not_narrowed_to_the_current_site asserts through a real WP_User_Query because the collision only shows up downstream of the filter.

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.47619% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.73%. Comparing base (74ad3e1) to head (8cee7a2).

Additional details and impacted files
@@                         Coverage Diff                         @@
##             feature/network-sites-column     #336       +/-   ##
===================================================================
+ Coverage                           84.55%   96.73%   +12.18%     
  Complexity                            188      188               
===================================================================
  Files                                  16       17        +1     
  Lines                                2356     2418       +62     
===================================================================
+ Hits                                 1992     2339      +347     
+ Misses                                364       79      -285     
Flag Coverage Δ
multisite 96.73% <90.47%> (?)
phpunit 82.38% <1.58%> (-2.17%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

Network Users Online view is overwritten by the single-site pre_get_users filter

1 participant