EES-5908 Add Event Raiser to Publisher #5694
Merged
Conversation
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
benoutram
reviewed
Mar 18, 2025
...oreEducationStatistics.Publisher/GovUk.Education.ExploreEducationStatistics.Publisher.csproj
Outdated
Show resolved
Hide resolved
src/GovUk.Education.ExploreEducationStatistics.Publisher/Options/EventGridOptions.cs
Outdated
Show resolved
Hide resolved
...ucation.ExploreEducationStatistics.Publisher/Services/EventGrid/IEventGridPublisherClient.cs
Show resolved
Hide resolved
...Uk.Education.ExploreEducationStatistics.Publisher/Services/Interfaces/IEventRaiserService.cs
Outdated
Show resolved
Hide resolved
src/GovUk.Education.ExploreEducationStatistics.Publisher/Services/EventRaiserService.cs
Outdated
Show resolved
Hide resolved
...vUk.Education.ExploreEducationStatistics.Publisher/Events/ReleaseVersionPublishedEventDto.cs
Show resolved
Hide resolved
src/GovUk.Education.ExploreEducationStatistics.Publisher/appsettings.json.tmpl
Outdated
Show resolved
Hide resolved
src/GovUk.Education.ExploreEducationStatistics.Publisher/PublisherHostBuilderExtensions.cs
Show resolved
Hide resolved
...cation.ExploreEducationStatistics.Publisher.Tests/Builders/Config/EventGridOptionsBuilder.cs
Outdated
Show resolved
Hide resolved
...ucation.ExploreEducationStatistics.Publisher.Tests/Builders/Services/UnitTestOutputLogger.cs
Show resolved
Hide resolved
benoutram
approved these changes
Mar 18, 2025
Happy Path done. Fix namespaces Methodology tests Test that latest release version is saved against the publication Content Service tests Notification, Redirects cache, and final publishing status tests If the release version being published is not the latest version, make sure the actual latest version is set on the publication Add DataSetPublishing tests Add Event raiser service to Publisher Code tidy Added Event Raiser implementation Fix merge issues Added EventRaiserService and an integration test to make sure it's working Forgot to mark the integration test as Skip Config tweaks
…erClient en route
leeoades
force-pushed
the
EES-5908-Add-event-raiser-to-publisher
branch
from
5088518 to
3743022
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Once the Publisher has finished performing all of its many publishing functions, it will raise an Event to signal any other interested parties that a Release Version has been published. This event contains related information about the release, the publication, associated slugs etc.
The first intended subscriber to this event is the function that creates a searchable document.