Skip to content

feat: push each site's online set into the network summary - #333

Open
josephfusco wants to merge 5 commits into
feature/network-summary-tablefrom
feature/network-summary-push
Open

feat: push each site's online set into the network summary#333
josephfusco wants to merge 5 commits into
feature/network-summary-tablefrom
feature/network-summary-push

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

The table in #332 has nothing writing into it.

Part of #298
Closes #327

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

github-actions Bot commented Aug 24, 2026

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 iamchitti, 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 6c1549be2d019605fd31de508cba208bb56a1f7c. Auto-updates when you push.

@josephfusco

josephfusco commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator Author

Review findings from #299 that land here are #310, answered by the debounce, plus #329 and #343, which are not fixed.

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.98%. Comparing base (6173f42) to head (6c1549b).

Additional details and impacted files
@@                         Coverage Diff                         @@
##             feature/network-summary-table     #333      +/-   ##
===================================================================
+ Coverage                            94.33%   96.98%   +2.65%     
  Complexity                             190      190              
===================================================================
  Files                                   15       15              
  Lines                                 2099     2159      +60     
===================================================================
+ Hits                                  1980     2094     +114     
+ Misses                                 119       65      -54     
Flag Coverage Δ
multisite 96.98% <96.66%> (?)
phpunit 92.07% <13.33%> (-2.26%) ⬇️

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-push branch from 8a3abdc to 4998a33 Compare August 24, 2026 20:36
@josephfusco
josephfusco force-pushed the feature/network-summary-push branch from 4998a33 to 0b06e38 Compare August 24, 2026 22:45
@josephfusco
josephfusco force-pushed the feature/network-summary-push branch from 0b06e38 to 6c1549b Compare August 24, 2026 22:54
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.

Deleting a user leaves their presence behind on every site they were online on

2 participants