Skip to content

Conversation

obenland
Copy link
Member

Fixes #1059.

Adds the required sync information to make like and repost notifications work for Jetpack and WoA sites.

Proposed changes:

  • Adds filters to include ActivityPub comment meta keys and custom comment types in Jetpack sync.

Other information:

  • Have you written new tests for your changes, if applicable?

Testing instructions:

  • Can only be tested in conjunction with changes on WordPress.com.

Changelog entry

  • Automatically create a changelog entry from the details below.
Changelog Entry Details

Significance

  • Patch
  • Minor
  • Major

Type

  • Added - for new features
  • Changed - for changes in existing functionality
  • Deprecated - for soon-to-be removed features
  • Removed - for now removed features
  • Fixed - for any bug fixes
  • Security - in case of vulnerabilities

Message

Updated sync allowlist to add support for Jetpack notifications of likes and reposts.

Adds filters to include ActivityPub comment meta keys and custom comment types in Jetpack sync. Specifically, 'avatar_url' is now whitelisted for comment meta sync, and comment types are registered earlier to ensure proper inclusion.
@obenland obenland requested a review from pfefferle September 24, 2025 20:19
@obenland obenland self-assigned this Sep 24, 2025
@Copilot Copilot AI review requested due to automatic review settings September 24, 2025 20:19
@github-actions github-actions bot added the [Focus] Compatibility Ensuring the plugin plays well with other plugins label Sep 24, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the Jetpack integration to enable ActivityPub comment synchronization for likes and repost notifications on Jetpack and WordPress.com sites.

  • Adds comment meta synchronization filtering to include ActivityPub-specific metadata
  • Implements comment type registration for proper sync handling during early WordPress hooks
  • Extends existing sync functionality to cover comment-related data alongside post data

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@pfefferle pfefferle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No idea how to test, but I fully trust you in everything what you do! 🤜🤛

@obenland obenland merged commit 905aaab into trunk Sep 25, 2025
20 of 22 checks passed
@obenland obenland deleted the update/interaction-notifications-sync branch September 25, 2025 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Focus] Compatibility Ensuring the plugin plays well with other plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Notifications for Interactions on Jetpack sites
3 participants