Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multisite context switch support to SyncManager #3503

Conversation

sathyapulse
Copy link
Contributor

@sathyapulse sathyapulse commented Jun 20, 2023

This commit adds support for Multisite context switching to the SyncManager class. This allows it to index the content to the correct site when indexing in the shutdown hook.

The commit introduces two methods in the SyncManager class: get_sync_queue and reset_sync_queue. The get_sync_queue method returns the sync queue for the current site or the $blog_id. The reset_sync_queue method resets the sync queue for the current site or the $blog_id.

This commit also updates all the dependent tests to use the new methods.

Fixes #3471 issue.

Description of the Change

Closes #3471

How to test the Change

Changelog Entry

Added - New feature
Changed - Existing functionality
Deprecated - Soon-to-be removed feature
Removed - Feature
Fixed - Bug fix
Security - Vulnerability

Credits

Props @username, @username2, ...

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

This commit adds support for Multisite context switching to the SyncManager class.
This allows it to index the content to the correct site when indexing in the shutdown hook.

The commit introduces two methods in the SyncManager class: `get_sync_queue` and `reset_sync_queue`.
The `get_sync_queue` method returns the sync queue for the current site or the `$blog_id`.
The `reset_sync_queue` method resets the sync queue for the current site or the `$blog_id`.

This commit also updates all the dependent tests to use the new methods.

Fixes 10up#3471 issue.
@felipeelia felipeelia added this to the 5.0.0 milestone Jul 4, 2023
@sathyapulse sathyapulse marked this pull request as ready for review August 8, 2023 12:16
@felipeelia
Copy link
Member

Closing in favor of #3689. Thanks @sathyapulse!

@felipeelia felipeelia closed this Oct 6, 2023
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.

BUG: Sync not working properly with multisite
2 participants