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.

Sign up for GitHub

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

Jump to bottom

Refactoring of the CQRS Flow .NET example based on the feedback #6

Merged
merged 2 commits into from Aug 5, 2021
Merged

Refactoring of the CQRS Flow .NET example based on the feedback #6

merged 2 commits into from Aug 5, 2021

Conversation

oskardudycz
Copy link
Contributor

@oskardudycz oskardudycz commented Aug 4, 2021 •
edited

  • Removed MediatR library not to have a redundant dependency from the external package (no advanced features like pipelines were used)
  • Removed marker interface for Commands, Queries and Events to reduce potential ceremony

@oskardudycz oskardudycz added cqrs enhancement New feature or request labels Aug 4, 2021
@oskardudycz
Removed MediatR library to not have redundant dependency from the ext…
bdfed61 
…ernal package (no advanced features like pipelines were used)
@oskardudycz oskardudycz marked this pull request as ready for review August 4, 2021 12:05
@mat-mcloughlin mat-mcloughlin merged commit 962416a into main Aug 5, 2021
@mat-mcloughlin mat-mcloughlin deleted the cqrs_flow/dotnet/refactoring branch August 5, 2021 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mat-mcloughlin mat-mcloughlin mat-mcloughlin approved these changes

@ylorph ylorph ylorph approved these changes

@alexeyzimarev alexeyzimarev Awaiting requested review from alexeyzimarev

Assignees
No one assigned
Labels
cqrs enhancement New feature or request .NET
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@oskardudycz @mat-mcloughlin @ylorph