Skip to content

Conversation

@acoumb
Copy link
Contributor

@acoumb acoumb commented Nov 11, 2025

Current PR attempts to fix issue #13 by forcing a memory cache reload when working in a load balanced environment through a ContentPublishedNotification handler.

@acoumb acoumb requested a review from Copilot November 11, 2025 12:38
Copilot finished reviewing on behalf of acoumb November 11, 2025 12:40
Copy link
Contributor

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 addresses issue #13 by implementing a memory cache reload mechanism for load-balanced environments in the Algolia search integration. The fix ensures that when content is published on a scheduling publisher server, the distributed cache is refreshed to maintain consistency across the environment.

  • Added a new notification handler for ContentPublishedNotification events
  • Version bumped to 2.3.3 across the project and package manifest
  • Registered the new handler in the composer

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
Umbraco.Cms.Integrations.Search.Algolia.csproj Updated package version from 2.3.2 to 2.3.3
AlgoliaContentPublishedHandler.cs New handler to trigger cache refresh on scheduling publisher when content is published
package.manifest Updated package version from 1.3.1 to 2.3.3
AlgoliaComposer.cs Registered the new ContentPublishedHandler and cleaned up whitespace

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@acoumb acoumb merged commit 1cb0bfa into main Nov 13, 2025
13 checks passed
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