Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Unify command handlers #85

Closed
lucabriguglia opened this issue Nov 1, 2019 · 5 comments
Closed

Unify command handlers #85

lucabriguglia opened this issue Nov 1, 2019 · 5 comments
Assignees
@lucabriguglia
Labels
breaking change enhancement
Milestone
Kledex 2.0

Comments

@lucabriguglia
Copy link
Owner

No description provided.

@lucabriguglia lucabriguglia added this to the Kledex 2.0 milestone Nov 1, 2019
@lucabriguglia lucabriguglia self-assigned this Nov 1, 2019
@lucabriguglia
Copy link
Owner Author

I'm getting rid of IDomainCommandHandler interface and all type of commands will use the ICommandHandler interface.

@andreujuanc
Copy link
Contributor

BRILLIANT.
I'm no longer working on the project in which we used Kledex. But this is a very welcome change.
More than once we had to debug code that was "not working" and the confusion was mostly on those interfaces/dispatchers.

@lucabriguglia
Copy link
Owner Author

Changes already in issue-85-command-handlers branch

@lucabriguglia
Copy link
Owner Author

I have also merged the command senders

@lucabriguglia
Copy link
Owner Author

Merged to release/2.0 branch

lucabriguglia pushed a commit that referenced this issue Nov 3, 2019
Merge #51 #82 #84 #85
1f27c7d
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@lucabriguglia lucabriguglia

Labels
breaking change enhancement
Projects
None yet
Milestone
Kledex 2.0
Development

No branches or pull requests
2 participants
@andreujuanc @lucabriguglia