Repository navigation
Groups: store check-ins in GatherPress's check-in flag - #2231
Merged
Merged
Conversation
GatherPress 0.36.0 has its own check-in, shown and changed on the wp-admin RSVPs screen. Ours lived in a comment meta, so neither side saw the other's check-ins. Read and write the `Check_In` flag instead. A hook on GatherPress's flag added/removed actions keeps our per-date check-in scopes in step, so a check-in made in wp-admin also switches "attended" over. `sync_scope()` filters the event's RSVPs by the flag in PHP rather than with a `tax_query`: `WP_Comment_Query` keys its cache on its own query vars only, so a `tax_query` query shares a cache entry with the same query without it and GatherPress's RSVP list came back holding only the checked-in RSVPs. The check-in time is dropped. The flag has no time, and nothing read it. `wp wc-groups-check-ins scan|migrate` moves existing check-ins onto the flag, one site at a time, and is deleted once it has run. Fixes #2213. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
bor0
marked this pull request as ready for review
October 22, 2026 11:08
bor0
added a commit
that referenced
this pull request
Oct 22, 2026
It ran across the groups network after #2231 deployed and found no check-ins to move, so nothing is left for it to do. Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #2213.
Check-ins now live in GatherPress's own
Check_Inflag instead of ourwporg_groups_checked_incomment meta, so the event page and the wp-admin RSVPs screen show and change the same thing.gatherpress_rsvp_flag_added/_removedkeeps our per-date check-in scopes in step, so a wp-admin check-in also switches "attended" over.wp wc-groups-check-ins scan|migratemoves existing check-ins. It's a one-off; delete it once run.Deploy
After deploying, on the events network:
Group sites are two path segments deep, so
--urlneeds a--requirethat returns 2 fromsite_by_path_segments_count. Untilmigrateruns on a site, its old check-ins don't show.Testing
wporg_groups_checked_inon an RSVP, runscan,migrate --dry-run,migrate, thenmigrateagain (nothing left).Screenshots
🤖 Generated with Claude Code