Skip to content

PC-8075 Updates to MessageDispatch Libraries#22

Merged
jbradford merged 34 commits intomasterfrom
PC-8075-LibraryUpdate
Jun 6, 2025
Merged

PC-8075 Updates to MessageDispatch Libraries#22
jbradford merged 34 commits intomasterfrom
PC-8075-LibraryUpdate

Conversation

@ChrisF-Pharmaxo
Copy link
Copy Markdown
Contributor

@ChrisF-Pharmaxo ChrisF-Pharmaxo commented May 28, 2025

The MessageDispatch library needed to be updated to use the latest version of KurrentDB to reference the Event Store GRPC client.. The work was primarily done in this library, with additional work being done in this PR: qphl/MessageDispatch.ProtobufSnapshotting#11

Work done as part of PC-8075

  • Renamed EventStore names to Kurrent.
  • Renamed eventstore folder to MessageDispatcher.KurrentDB
  • Added C# code styles / formatting
  • Updated packages.
  • Changed Corsham references to Pharmaxo.
  • Updated Subscription logic to reflect the way KurrentDB advises implementing it in their current documents: https://docs.kurrent.io/clients/grpc/subscriptions.html#handling-subscription-drops.
  • Multi-targetting .NET 4.8.1 and .NET 8
  • Adding test coverage to the library (Todo)

Ticket: https://qphl.atlassian.net/browse/PC-8075

@danieljayne danieljayne marked this pull request as ready for review June 2, 2025 09:46
Copy link
Copy Markdown
Contributor

@JoshPattie JoshPattie left a comment

Choose a reason for hiding this comment

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

Just one comment. I think this has been thoroughly reviewed at this point!

PC-8075 Library Update (Testcontainer Tests)
@jbradford jbradford merged commit 0b62c5d into master Jun 6, 2025
4 checks passed
@jbradford jbradford deleted the PC-8075-LibraryUpdate branch June 6, 2025 14:57
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.

5 participants