Use event start date for speaker badge activity timestamp#1662
Open
dd32 wants to merge 2 commits intoproductionfrom
Open
Use event start date for speaker badge activity timestamp#1662dd32 wants to merge 2 commits intoproductionfrom
dd32 wants to merge 2 commits intoproductionfrom
Conversation
When a speaker badge is awarded later (e.g. months after the event when the WordPress.org username is finally added), the activity timestamp showed the current date instead of when the event actually happened. Now uses the WordCamp start date for the activity timestamp, falling back to the post published date. This ensures the badge activity appears at the correct point in the user timeline. Fixes #1432 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
For events that have not yet occurred, use the current time instead of the future event start date. This prevents activity entries from having timestamps in the future. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
3 tasks
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
1 participant
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.
Summary
get_post_activity_payload()frompost_modified_gmtto the WordCamp start datepost_date_gmt(published date) if no event start date is availableChanges
wordcamp-participation-notifier.php: Replacestrtotime($post->post_modified_gmt)with WordCamp start date, falling back to post published dateTest plan
Fixes #1432
🤖 Generated with Claude Code