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

Display sync trigger on sync page. #3722

Merged
merged 5 commits into from
Oct 24, 2023
Merged

Display sync trigger on sync page. #3722

merged 5 commits into from
Oct 24, 2023

Conversation

JakePT
Copy link
Contributor

@JakePT JakePT commented Oct 24, 2023

Description of the Change

Displays a message indicating what triggered a sync on the sync page.

How to test the Change

  • When syncing from the Sync page, the sync progress should read "Started manually from the Sync page"
  • When syncing after saving feature settings, the sync progress should read "Started automatically after a change to feature settings"
  • When syncing after installing, the sync progress should read "Started automatically after installing the ElasticPress plugin"
  • The Sync History entries should have shorter versions of the same messages.

Changelog Entry

Added - The sync page now describes what triggered the current sync, and previous syncs.

Credits

Props @JakePT

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.

@felipeelia felipeelia merged commit a858325 into 5.0.0 Oct 24, 2023
13 checks passed
@felipeelia felipeelia deleted the feature/sync-trigger branch October 24, 2023 19:48
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.

2 participants