Skip to content

feat: read the network summary as a capped snapshot - #334

Open
josephfusco wants to merge 1 commit into
feature/network-summary-pushfrom
feature/network-summary-read
Open

feat: read the network summary as a capped snapshot#334
josephfusco wants to merge 1 commit into
feature/network-summary-pushfrom
feature/network-summary-read

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 (this one)
  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
  6. feat: add a Who's Online widget to the Network Admin dashboard #337 network dashboard widget

The rows #333 writes have nothing reading them, and #310 asked whether that read holds up at large-network scale.

Part of #298
Closes #321
Closes #324

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 c323b1fbdc5373ae9ab77e2bca4e33614b66942f. Auto-updates when you push.

@josephfusco

josephfusco commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator Author

Review findings from #299 and #313 that land here are #321 and #324, both closed by this one, with #327, #328, and #344 still open against the same code.

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.42857% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.91%. Comparing base (0b06e38) to head (129e066).

Additional details and impacted files
@@                        Coverage Diff                         @@
##             feature/network-summary-push     #334      +/-   ##
==================================================================
+ Coverage                           92.07%   96.91%   +4.83%     
  Complexity                            190      190              
==================================================================
  Files                                  15       15              
  Lines                                2159     2334     +175     
==================================================================
+ Hits                                 1988     2262     +274     
+ Misses                                171       72      -99     
Flag Coverage Δ
multisite 96.91% <95.42%> (?)
phpunit 85.17% <0.00%> (-6.91%) ⬇️

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.

@josephfusco
josephfusco force-pushed the feature/network-summary-read branch from c50236f to c1cb292 Compare August 24, 2026 20:36
@josephfusco
josephfusco force-pushed the feature/network-summary-read branch from c1cb292 to 129e066 Compare August 24, 2026 22:45
@josephfusco
josephfusco force-pushed the feature/network-summary-read branch from 129e066 to c323b1f Compare August 24, 2026 22:54
@josephfusco josephfusco mentioned this pull request Aug 24, 2026
8 tasks
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 summary read path is unbounded Network summary is rebuilt once per list table row

1 participant